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

Issue#48 remove ingress externalport parameter #53

Conversation

ridoo
Copy link
Contributor

@ridoo ridoo commented Aug 9, 2023

Description

Remove unnecessary geonode.ingress.externalPort parameter. Both http schemes http and https have their default ports 80 and 443 which should not be configured explicitly.

ref #48

Type of Change

Please select the relevant option:

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • Other (please describe)

Related Issue

If there is an existing issue related to this pull request, please reference it here.

closes #48

Checklist

Please ensure that your pull request meets the following requirements:

  • The pull request is limited to one type (docs, feature, bug fix, etc.)
  • The pull request is as small as possible. Consider opening multiple pull requests instead of one large one.
  • The feature or bug fix has been discussed and documented in an issue beforehand.

@mwallschlaeger mwallschlaeger merged commit f907a8a into zalf-rdm:main Aug 10, 2023
2 checks passed
@ridoo ridoo deleted the issue#48_remove-ingress-externalport-parameter branch September 10, 2023 16:46
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.

Task: Remove ingress externalPort parameter
2 participants