Skip to content

Update default VOMS AA application.yml#919

Open
jouvin wants to merge 1 commit intoindigo-iam:developfrom
jouvin:voms-aa_application_yml
Open

Update default VOMS AA application.yml#919
jouvin wants to merge 1 commit intoindigo-iam:developfrom
jouvin:voms-aa_application_yml

Conversation

@jouvin
Copy link

@jouvin jouvin commented Feb 6, 2025

This PR replaces the default VOMS AA application.yml by a version using environment variables to configure it, to avoid a site the need to deploy its own specific version. The variables used are those already used by the login service plus a few ones specific to the VOMS AA service:

  • VOMS_AA_HOST: the host where VOMS AA service is running (no default)
  • VOMS_AA_PORT: the port the VOMS AA service binds to (no default)
  • VOMS_AA_VO: the VO served by the VOMS AA service (no default)
  • VOMS_AA_USE_LEGACY_FQAN_ENCODING: defaults to true

This change should not have any impact on existing VOMS AA instances as the current default application.yml is not usable, meaning that sites will have deployed their own specific version that will continue to override the default one.

  • Once/if this PR is accepted, the application.yml contents to use (similar to this one) should be removed from the VOMS AA documentation.

@jouvin jouvin force-pushed the voms-aa_application_yml branch from 3994dde to de587b1 Compare February 6, 2025 08:45
@jouvin jouvin changed the base branch from master to develop February 6, 2025 08:47
@jouvin jouvin force-pushed the voms-aa_application_yml branch from de587b1 to 0ded8c2 Compare February 6, 2025 08:48
jouvin added a commit to jouvin/iam-website that referenced this pull request Feb 6, 2025
@jouvin jouvin force-pushed the voms-aa_application_yml branch from 0ded8c2 to dc8f409 Compare February 19, 2026 12:07
- Current version cannot be used out of the shelf
- Be consistent with variable already used for the login service
- Add variables VOMS_AA_HOST, VOMS_AA_PORT, VOMS_AA_VO and VOMS_AA_USE_LEGACY_FQAN_ENCODING specific to VOMS AA
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