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

fix incorrect geoserver URL in cowbird configuration #487

Merged
merged 4 commits into from
Dec 18, 2024

Conversation

mishaschwartz
Copy link
Collaborator

Overview

The Geoserver URL in the Cowbird configuration was incorrect as it used a non-standard port.

This change removes the port so that the URL will resolve properly. This allows Cowbird to execute webhooks that target Geoserver.

Changes

Non-breaking changes

  • Bug fix

Breaking changes

  • None

Related Issue / Discussion

Additional Information

Links to other issues or sources.

CI Operations

birdhouse_daccs_configs_branch: master
birdhouse_skip_ci: false

@github-actions github-actions bot added component/cowbird Related to https://github.com/Ouranosinc/cowbird documentation Improvements or additions to documentation labels Dec 13, 2024
Copy link
Collaborator

@tlvu tlvu left a comment

Choose a reason for hiding this comment

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

LGTM as I do not really know how Cowbird integrate with Geoserver.

CHANGES.md Outdated Show resolved Hide resolved
@github-actions github-actions bot added the ci/operations Continuous Integration components label Dec 18, 2024
@mishaschwartz mishaschwartz merged commit 020522b into master Dec 18, 2024
4 of 5 checks passed
@mishaschwartz mishaschwartz deleted the fix-cowbird-geoserver-interaction branch December 18, 2024 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/operations Continuous Integration components component/cowbird Related to https://github.com/Ouranosinc/cowbird documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants