Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

webapi-docker profile still uses non-TLS url #2230

Closed
alondhe opened this issue Mar 16, 2023 · 1 comment
Closed

webapi-docker profile still uses non-TLS url #2230

alondhe opened this issue Mar 16, 2023 · 1 comment

Comments

@alondhe
Copy link
Contributor

alondhe commented Mar 16, 2023

Expected behavior

As discussed here: #1825 we should use TLS based URLs in the pom.

Actual behavior

This is not the case in the webapi-docker profile:

https://github.com/OHDSI/WebAPI/blob/master/pom.xml#L1345

Steps to reproduce behavior

As we lack the port access for 8085, this causes Docker builds to fail in our site.
@alondhe alondhe mentioned this issue Mar 16, 2023
Closed
@alondhe
Copy link
Contributor Author

alondhe commented Mar 21, 2023

Additionally, the webapi-docker profile appears to be using an "ohdsi.snapshots" repo for several OHDSI dependencies.

@alondhe alondhe mentioned this issue Mar 27, 2023
Merged
pieterlukasse pushed a commit to pieterlukasse/WebAPI that referenced this issue Sep 6, 2023
@alondhe @pieterlukasse
Webapi docker profile and TLS (OHDSI#2235)
31c1935 
* Changed non-TLS nexus to use TLS
* Removed duplicate repo entries; added ohdsi.snapshot entry

Fixes OHDSI#2230.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@alondhe