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

Configure Harbor with HTTPS Access #44

Merged
merged 4 commits into from
Mar 29, 2016
Merged

Conversation

xiahaoshawn
Copy link
Contributor

add cert directory and nginx.https.conf file to Deploy/config/nginx/

}
server {
listen 80;
server_name harbordomian.com;
Copy link
Contributor

Choose a reason for hiding this comment

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

harbordomain.com

@xiahaoshawn
Copy link
Contributor Author

docker-compose.yml need change proxy volume and port sections
volumes:
- ./config/nginx/nginx.https.conf:/etc/nginx/nginx.conf
- ./config/nginx/cert:/etc/nginx/cert
ports:
- 80:80
- 443:443

@xiaods
Copy link
Contributor

xiaods commented Mar 26, 2016

@xiahaoshawn we need update a doc to guide user.

@xiahaoshawn xiahaoshawn changed the title add cert directory and nginx.https.conf file Configure Harbor with HTTPS Access Mar 28, 2016
@reasonerjt reasonerjt merged commit 99e2774 into goharbor:master Mar 29, 2016
supereagle pushed a commit to supereagle/harbor that referenced this pull request Mar 5, 2019
Save image vulnerability into DB
bitsf pushed a commit to bitsf/harbor that referenced this pull request Mar 22, 2022
…INOR-fix-chartmuseum to master

* commit 'ecd3f40d14ab5850871c184c1b5cf14df30945a1':
  fix(chartmuseum) Add CACHE_URL environment variable only if cache secret is specifed
bupd pushed a commit to bupd/harbor that referenced this pull request Oct 21, 2024
Adds the registry URL to the state artifact file instead of its name
bupd pushed a commit to bupd/harbor that referenced this pull request Oct 21, 2024
Adds the registry URL to the state artifact file instead of its name
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.

3 participants