Skip to content

Fixing the issues with the reverse proxy #242

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

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Conversation

cjlapao
Copy link
Collaborator

@cjlapao cjlapao commented Nov 18, 2024

Description

  • Fixed an issue where the reverse proxy would fail to initialize due to prot conflict and any attempt to restart would crash
  • fixed an issue where we allowed routes to be created even if the reverse proxy was disabled
  • fixed an issue where if the config changes in memory we would not save those changes
  • added the ability to choose the port and host trough configuration or env variables

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have run tests that prove my fix is effective or that my feature works
  • I have updated the CHANGELOG.md file accordingly

- Fixed an issue where the reverse proxy would fail to initialize due to prot conflict and any attempt to restart would crash
- fixed an issue where we allowed routes to be created even if the reverse proxy was disabled
- fixed an issue where if the config changes in memory we would not save those changes
- added the ability to choose the port and host trough configuration or env variables
@cjlapao cjlapao added this to the 0.9.12 milestone Nov 18, 2024
@cjlapao cjlapao merged commit 1a9730b into main Nov 18, 2024
12 checks passed
@cjlapao cjlapao deleted the cjlapao/hotfix-reverse-proxy branch November 18, 2024 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

1 participant