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

upgrade to 0.68.0 #11

Merged
merged 1 commit into from
Oct 30, 2024
Merged

upgrade to 0.68.0 #11

merged 1 commit into from
Oct 30, 2024

Conversation

cedricve
Copy link
Member

@cedricve cedricve commented Oct 30, 2024

Upgrade to 0.68.0

Motivation

The primary motivation behind this upgrade is to keep the project up-to-date with the latest version, ensuring we benefit from the latest features, improvements, and bug fixes. Upgrading to version 0.68.0 helps maintain the project's stability, security, and performance.

Changes

  1. Chart Version Update:

    • Updated the chart version in charts/hub/Chart.yaml from 0.67.0 to 0.68.0.
  2. Configuration Adjustments:

    • Moved the nginx.ingress.kubernetes.io/enable-cors annotation from under the conditional block for kerberoshub.oauth2Proxy.enabled to the tls-acme block in charts/hub/templates/kerberos-hub/hub-api.yaml. This ensures that CORS is enabled only when TLS is used, improving security.

Benefits

  • Latest Features and Fixes: By upgrading to the latest version, we ensure that the project incorporates the newest features and bug fixes.
  • Improved Security: Keeping dependencies and configurations up-to-date minimizes potential security vulnerabilities.
  • Configuration Optimization: The adjustment to the CORS configuration ensures that it is applied more appropriately, enhancing security and functionality.

Overall, this upgrade is a necessary step to maintain the project's health and ensure it continues to function optimally.

@cedricve cedricve self-assigned this Oct 30, 2024
@cedricve cedricve merged commit 19d3f0c into main Oct 30, 2024
2 checks passed
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.

1 participant