Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[py3] REST/Server.py - encode to bytes input of hashlib.sha1 #10917

Merged
merged 1 commit into from
Dec 15, 2021

Conversation

mapellidario
Copy link
Member

Fixes #10916

Status

Ready, tested in CRABServer test and preprod instances

This webpage https://cmsweb-testbed.cern.ch/crabserver/ui/ is being server by a build of CRABserver that contiains this fix: https://github.com/mapellidario/WMCore/releases/tag/py3.211209

Since we are building CRABServer against our private repo, which contains this fix, we do not have any hurry in mergins this PR.

Description

Make sure that the input of hashlib.sha1() is bytes.

Is it backward compatible (if not, which system it affects?)

yes

Related PRs

none

External dependencies / deployment changes

none

@cmsdmwmbot
Copy link

Jenkins results:

  • Python2 Unit tests: succeeded
  • Python3 Unit tests: succeeded
    • 1 changes in unstable tests
  • Python2 Pylint check: failed
    • 50 warnings and errors that must be fixed
    • 20 warnings
    • 84 comments to review
  • Python3 Pylint check: failed
    • 55 warnings and errors that must be fixed
    • 20 warnings
    • 90 comments to review
  • Pylint py3k check: succeeded
  • Pycodestyle check: succeeded
    • 6 comments to review

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-WMCore-PR-test/12612/artifact/artifacts/PullRequestReport.html

Copy link
Contributor

@amaltaro amaltaro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again, Dario! It looks good to me (honestly, I thought I have had a PR fixing those throughout WMCore...)

@amaltaro amaltaro merged commit 00933bd into dmwm:master Dec 15, 2021
@mapellidario
Copy link
Member Author

honestly, I thought I have had a PR fixing those throughout WMCore..
Yeah! I do not know how and why missed this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

REST/Server.py - improve string support for ETag
3 participants