Skip to content

fix: allow HTTPS for localhost#1107

Merged
qweeah merged 10 commits into
oras-project:mainfrom
qweeah:qweeah/fix-plain-http
Sep 11, 2023
Merged

fix: allow HTTPS for localhost#1107
qweeah merged 10 commits into
oras-project:mainfrom
qweeah:qweeah/fix-plain-http

Conversation

@qweeah

@qweeah qweeah commented Sep 7, 2023

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
This PR allows user to specify --plain-http=false for a registry named localhost

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1106

Please check the following list:

  • Does the affected code have corresponding tests, e.g. unit test, E2E test?
  • Does this change require a documentation update?
  • Does this introduce breaking changes that would require an announcement or bumping the major version?
  • Do all new files have an appropriate license header?

Signed-off-by: Billy Zha <jinzha1@microsoft.com>
@qweeah qweeah force-pushed the qweeah/fix-plain-http branch from 34fcb7e to 4f290cc Compare September 7, 2023 07:23
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
@qweeah qweeah marked this pull request as ready for review September 7, 2023 07:40
@codecov-commenter

codecov-commenter commented Sep 7, 2023

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.37%. Comparing base (ed1a9e6) to head (dc3c297).
⚠️ Report is 544 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1107      +/-   ##
==========================================
+ Coverage   80.31%   80.37%   +0.06%     
==========================================
  Files          57       57              
  Lines        2931     2940       +9     
==========================================
+ Hits         2354     2363       +9     
  Misses        398      398              
  Partials      179      179              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread cmd/oras/internal/option/remote.go
Comment thread cmd/oras/internal/option/remote.go Outdated
Comment thread cmd/oras/internal/option/remote.go Outdated
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
Comment thread cmd/oras/internal/option/remote.go Outdated
Signed-off-by: Billy Zha <jinzha1@microsoft.com>
@qweeah qweeah requested a review from shizhMSFT September 11, 2023 04:25

@shizhMSFT shizhMSFT left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@qweeah qweeah merged commit 7a822cc into oras-project:main Sep 11, 2023
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.

oras tries to use http instead of https

3 participants