New issue
Advanced search Search tips

Issue 845532 link

Starred by 2 users
Status: Fixed
Owner:
valleau@chromium.org
Closed: May 2018
Cc:
skau@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Change HTTP_ENCYRPTION_REQUIRED to HTTP_ENCRYPTION_ALWAYS for ipps

Project Member Reported by skau@chromium.org, May 22 2018 
HTTP_ENCRYPTION_REQUIRED requests a TLS upgrade where HTTP_ENCRYPTION_ALWAYS establishes a TLS connection immediately.  Some ipps endpoints reject the upgrade request.

Fix needed in:
https://cs.chromium.org/chromium/src/printing/backend/cups_jobs.cc?type=cs&sq=package:chromium&g=0

Comment 1 by valleau@chromium.org, May 23 2018 

I've sent out a change for review that should fix this issue.

Setup now works successfully on the following Canon devices:
Canon D570
Canon MX490 Series
Canon MF720C Series

Also tested on the following non-Canon devices to ensure I didn't break anything:
HP Envy 4520
Epson XP-640 Series
Project Member

Comment 2 by bugdroid1@chromium.org, May 23 2018 

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

commit 79b699eb40ca1860b886887b36e442980e02d5cb
Author: David Valleau <valleau@chromium.org>
Date: Wed May 23 01:33:01 2018

Changing ipps "HTTP_ENCRYPTION_REQUIRED" to "HTTP_ENCRYPTION_ALWAYS"

BUG= chromium:845532 
R=skau@chromium.org

Change-Id: I320f992f3f427d2a2675954eb59fc05e2a5c6a90
Reviewed-on: https://chromium-review.googlesource.com/1069837
Reviewed-by: Sean Kau <skau@chromium.org>
Commit-Queue: David Valleau <valleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560877}
[modify] https://crrev.com/79b699eb40ca1860b886887b36e442980e02d5cb/printing/backend/cups_jobs.cc

Comment 3 by valleau@chromium.org, May 23 2018

Status: Fixed (was: Assigned)
►

Sign in to add a comment