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

Issue 640645 link

Starred by 2 users
Status: WontFix
Owner: ----
Closed: Feb 2018
Cc:
█ n...@chromium.org
█ culli...@amazon.com
yfried...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

.../SSLClientCertificateRequest.java:200 Must be one of:[...] WrongConstant [warning]

Project Member Reported by mcasas@chromium.org, Aug 24 2016 

/tmp/tmpuJsWpS/SRC_ROOT1/chrome/android/java/src/org/chromium/chrome/browser/SSLClientCertificateRequest.java:200 Must be one of: KeyProperties.KEY_ALGORITHM_RSA, KeyProperties.KEY_ALGORITHM_EC, KeyProperties.KEY_ALGORITHM_AES, KeyProperties.KEY_ALGORITHM_HMAC_SHA1, KeyProperties.KEY_ALGORITHM_HMAC_SHA224, KeyProperties.KEY_ALGORITHM_HMAC_SHA256, KeyProperties.KEY_ALGORITHM_HMAC_SHA384, KeyProperties.KEY_ALGORITHM_HMAC_SHA512: WrongConstant [warning]
            KeyChain.choosePrivateKeyAlias(mActivity, mCallback, mKeyTypes, mPrincipalsForCallback,
                                                                 ~~~~~~~~~

https://cs.chromium.org/chromium/src/chrome/android/java/src/org/chromium/chrome/browser/SSLClientCertificateRequest.java?q=SSLClientCertificateRequest.java&sq=package:chromium&dr&l=200

seen in e.g.
https://build.chromium.org/p/chromium.linux/builders/Android%20Builder%20%28dbg%29/builds/111784/steps/compile/logs/stdio/text

landed in https://codereview.chromium.org/1327103003
Project Member

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

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 2 by cmasso@google.com, Feb 14 2018

Components: Internals>Network>SSL

Comment 3 by davidben@chromium.org, Feb 14 2018

Status: WontFix (was: Untriaged)
I'm not able to reproduce this, and the warning is wrong. We do indeed pass one of those values in. It comes from the C++ side of things. I'm going to assume this is a bug in whatever analyzer this is that has since been fixed, unless there is a way to repro this that I'm just missing.
►

Sign in to add a comment