New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 754305 link

Starred by 10 users

Chrome printing bug: Can't change to secondary user's printers

Reported by eric.ein...@gmail.com, Aug 10 2017

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/60.0.3112.90 Safari/537.36

Steps to reproduce the problem:
1. Print from browser (ctrl+P)
2. Click "Change" under Destination
3. Choose another user from dropdown box for Showing printers for.

What is the expected behavior?
Should list the other user's printers.

What went wrong?
Dropdown box reverts to first login and printers. Other user's printers are no longer accessible.

Did this work before? Yes about 10-20 days ago.

Chrome version: 60.0.3112.90  Channel: stable
OS Version: 10.0
Flash Version: Shockwave Flash 26.0 r0
 
Cc: jayhlee@google.com marcore@chromium.org
Components: Services>CloudPrint Internals>Printing
Labels: -Type-Bug-Regression Hotlist-Enterprise M-60 M-62 OS-Linux Type-Bug
Status: Untriaged (was: Unconfirmed)
Summary: Chrome printing bug: Can't change to secondary user's printers (was: Can't change to secondary user's printers)
I'm able to reproduce on all version of chrome that I've installed on my windows test machine:
Google Chrome	44.0.2403.89 (Official Build) (64-bit)
Google Chrome	56.0.2924.87 (Official Build) (64-bit)
Google Chrome	58.0.3029.110 (Official Build) (64-bit)
Google Chrome	59.0.3071.115 (Official Build) (64-bit)
Google Chrome	60.0.3112.90 (Official Build) (64-bit) (cohort: Stable)
Google Chrome	62.0.3182.0 (Official Build) canary (64-bit) (cohort: Clang-64)

also on linux:
Google Chrome	60.0.3112.90 (Official Build) (64-bit)

video: https://drive.google.com/open?id=0B01ZVp8vDQocd3BweUo5c3oxNEE
net-export: https://drive.google.com/open?id=0B01ZVp8vDQocSTRpd19xWEx2bkU
fiddler log: https://drive.google.com/open?id=0B01ZVp8vDQocZTNlVUtoVWkxZFE

Owner: bustamante@chromium.org
Hi richard, could you please help triage this issue ?

Components: -Internals>Printing UI>Browser>PrintPreview
Have been looking into this from the print preview side (discovered this issue a couple days ago). In case it helps, it looks like the search request is being sent for the correct user at [1] but the result that is returned at [2] is always for the original user. 
[1] https://cs.chromium.org/chromium/src/chrome/browser/resources/print_preview/cloud_print_interface.js?l=209
[2] https://cs.chromium.org/chromium/src/chrome/browser/resources/print_preview/cloud_print_interface.js?l=517
Cc: bustamante@chromium.org
Owner: paolof@chromium.org
Status: Assigned (was: Untriaged)
Given that this changed recently but all Chrome builds are behaving the same, plus the findings in comment 3 (seems like print preview is sending the request correctly and getting an incorrect response from the server), assigning to paolof@ from cloud print. Feel free to send it back if it turns out there is something wrong on the print preview side, but at this point this looks like a server side issue.
 Issue 755189  has been merged into this issue.

Comment 6 by paolof@chromium.org, Aug 15 2017

Cloud print server has been forced to switch from URL query parameter "user" to URL parameter "authuser" to manage multilogin situation. So chrome very likely still uses the "user" parameter, but the server ignores it.

This is not our fault. We have been forced to do that. If I knew that chrome had multilogin support for printing, we could have coordinated the timing of the transition.

Unfortunately we cannot revert to the previous behavior. It is impossible. But I can see two solutions:
1) Chrome starting using "authuser" parameter instead of "user".
2) If chrome correctly follows 302 redirect response even for POST requests, the server could return a 302 that replaces a "user" parameter with "authuser".

The first is a long term fix, which requires time. The second is a short-term fix, which could be implemented in one week. I'd implement both.

Comments or opinions?

Comment 7 by paolof@chromium.org, Aug 15 2017

Cc: bblietz@chromium.org

Comment 8 by paolof@chromium.org, Aug 15 2017

Cc: thestig@chromium.org adlr@chromium.org
Paolo can you please update this with the latest status and assign to Lei if appropriate?
Actually, I see your update in #6 now.  
I'll assign to Lei for comment.
Owner: thestig@chromium.org
Labels: Hotlist-ConOps
Owner: rbpotter@chromium.org
Cc: msrchandra@chromium.org ranjitkan@chromium.org rbasuvula@chromium.org nyerramilli@chromium.org
 Issue 757754  has been merged into this issue.

Comment 16 by adlr@chromium.org, Aug 22 2017

Lei, can you comment on Comment #6?
We emailed paolof@ yesterday to discuss solution 1.
Project Member

Comment 18 by bugdroid1@chromium.org, Aug 23 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/116a3c96c5f41515cca1f6d19f5935157a5f8d7b

commit 116a3c96c5f41515cca1f6d19f5935157a5f8d7b
Author: rbpotter <rbpotter@chromium.org>
Date: Wed Aug 23 02:35:03 2017

Use authuser instead of user in cloudprint queries

Cloudprint has changed to use authuser instead of user for queries.
Update the print preview code to match.

Bug:  754305 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I66258fa3776a0102560961bdd1e7e99e6c0d4dcc
Reviewed-on: https://chromium-review.googlesource.com/627069
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#496565}
[modify] https://crrev.com/116a3c96c5f41515cca1f6d19f5935157a5f8d7b/chrome/browser/resources/print_preview/cloud_print_interface.js
[modify] https://crrev.com/116a3c96c5f41515cca1f6d19f5935157a5f8d7b/chrome/browser/resources/print_preview/print_preview.js
[modify] https://crrev.com/116a3c96c5f41515cca1f6d19f5935157a5f8d7b/chrome/browser/ui/webui/print_preview/print_preview_handler.cc

Labels: TE-Verified-M62 TE-Verified-62.0.3196.0
Verified this issue on Windows 10, Ubuntu 14.04 with chrome #62.0.3196.0 
Observed that printer's list is displayed according to the user's availability, Hence adding TE-Verified labels.

Screen-Cast link : https://drive.google.com/a/google.com/file/d/0BywU0GRwmpVvM05fdjFJTE1ab0k/view?usp=sharing
Labels: M-61 Merge-Request-61 OS-Chrome OS-Mac
We'll need this for M61.
Project Member

Comment 21 by sheriffbot@chromium.org, Aug 25 2017

Labels: -Merge-Request-61 Merge-Review-61 Hotlist-Merge-Review
This bug requires manual review: We are only 10 days from stable.
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: pchalla@chromium.org
This merge is needed to update Chrome (the client) to match the expectations on the Cloud Print server side. Since the server side changes are not synced with Chrome's release schedule, and we didn't get an advanced notice, merges are unavoidable. The code to be merged only affects cloud printing.
Labels: -Merge-Review-61 Merge-Approved-61
Approving merge to M61 branch 3163 based on comment #22. Please merge ASAP. Thank you.
Project Member

Comment 24 by bugdroid1@chromium.org, Aug 25 2017

Labels: -merge-approved-61 merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5890f7f464800fbc9b8d9689f0fe071b84cad08a

commit 5890f7f464800fbc9b8d9689f0fe071b84cad08a
Author: rbpotter <rbpotter@chromium.org>
Date: Fri Aug 25 21:21:25 2017

Use authuser instead of user in cloudprint queries

Cloudprint has changed to use authuser instead of user for queries.
Update the print preview code to match.

TBR=rbpotter@chromium.org

(cherry picked from commit 116a3c96c5f41515cca1f6d19f5935157a5f8d7b)

Bug:  754305 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I66258fa3776a0102560961bdd1e7e99e6c0d4dcc
Reviewed-on: https://chromium-review.googlesource.com/627069
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Rebekah Potter <rbpotter@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#496565}
Reviewed-on: https://chromium-review.googlesource.com/636448
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#903}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/5890f7f464800fbc9b8d9689f0fe071b84cad08a/chrome/browser/resources/print_preview/cloud_print_interface.js
[modify] https://crrev.com/5890f7f464800fbc9b8d9689f0fe071b84cad08a/chrome/browser/resources/print_preview/print_preview.js
[modify] https://crrev.com/5890f7f464800fbc9b8d9689f0fe071b84cad08a/chrome/browser/ui/webui/print_preview/print_preview_handler.cc

Labels: TE-Verified-61.0.3163.71 TE-Verified-M61
Tested the issue on Windows-7 & 10, Ubuntu 14.04 and Mac OS 10.12.6 using chrome latest Beta M61-61.0.3163.71 by following steps mentioned in the original comment. Observed that printer's list displayed according to the user's availability and its working as expected. Hence adding TE-Verified label.

Please find the screen shot(Linux) for reference.

Thank you!
754305.ogv
3.2 MB View Download
Status: Fixed (was: Assigned)
Issue 758741 has been merged into this issue.
 Issue 752584  has been merged into this issue.

Sign in to add a comment