Chrome printing bug: Can't change to secondary user's printers
Reported by
eric.ein...@gmail.com,
Aug 10 2017
|
||||||||||||||||||
Issue descriptionUserAgent: 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
,
Aug 11 2017
Hi richard, could you please help triage this issue ?
,
Aug 11 2017
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
,
Aug 12 2017
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.
,
Aug 14 2017
Issue 755189 has been merged into this issue.
,
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?
,
Aug 15 2017
,
Aug 15 2017
,
Aug 15 2017
It looks like we're looking for the user parameter in the javascript. https://cs.chromium.org/chromium/src/chrome/browser/resources/print_preview/cloud_print_interface.js?rcl=411fb85ffd1c2478c78e9f6e6105d4c88f38ad96&l=444 Also, requests are sent via xhr, potentially limiting options for mitigation: https://cs.chromium.org/chromium/src/chrome/browser/resources/print_preview/cloud_print_interface.js?rcl=411fb85ffd1c2478c78e9f6e6105d4c88f38ad96&l=405
,
Aug 18 2017
Paolo can you please update this with the latest status and assign to Lei if appropriate?
,
Aug 18 2017
Actually, I see your update in #6 now. I'll assign to Lei for comment.
,
Aug 18 2017
,
Aug 18 2017
,
Aug 21 2017
,
Aug 22 2017
Issue 757754 has been merged into this issue.
,
Aug 22 2017
Lei, can you comment on Comment #6?
,
Aug 22 2017
We emailed paolof@ yesterday to discuss solution 1.
,
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
,
Aug 25 2017
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
,
Aug 25 2017
We'll need this for M61.
,
Aug 25 2017
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
,
Aug 25 2017
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.
,
Aug 25 2017
Approving merge to M61 branch 3163 based on comment #22. Please merge ASAP. Thank you.
,
Aug 25 2017
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
,
Aug 30 2017
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!
,
Sep 9 2017
,
Sep 19 2017
Issue 758741 has been merged into this issue.
,
Mar 2 2018
Issue 752584 has been merged into this issue. |
||||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||||
Comment 1 by marcore@chromium.org
, Aug 11 2017Components: 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)