Skip to content

Commit

Permalink
Fixed setup_x509 to be successful w/ TRUST_DOMAIN set in the as Facto…
Browse files Browse the repository at this point in the history
…ry or Frontend parameter

Includes CHANGELOG entries from other PRs
  • Loading branch information
mambelli committed Sep 27, 2023
1 parent f0065cc commit 0c6f072
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,11 @@ Bug fix quick release

### Bug Fixes

- Workaround for EL7 PyJWT bug, generating bytes instead of str (PR #355)
- Added bash requirement to files using bashisms, notably `glidein_sitewms_setup.sh` (PR #358)
- Fixed syntax errors in analyze_queues (PR #357)
- Fixed setup_x509 to be successful w/ TRUST_DOMAIN set in the as Factory or Frontend parameter (PR #359)
- GLIDEIN_SINGULARITY_BINARY_OVERRIDE set also with Frontend and Factory params, not only WN environment (PR #360)

### Testing / Development

Expand Down
4 changes: 3 additions & 1 deletion creation/web_base/setup_x509.sh
Original file line number Diff line number Diff line change
Expand Up @@ -441,7 +441,9 @@ _main() {
gconfig_add GLIDEIN_CONDOR_TOKEN "$GLIDEIN_CONDOR_TOKEN"
gconfig_add GLIDEIN_CONDOR_TOKEN_ORIG "$GLIDEIN_CONDOR_TOKEN_ORIG"
out=$(gconfig_get TRUST_DOMAIN "${glidein_config}")
if [[ -z "$out" ]]; then
if [[ -n "$out" ]]; then
cred_updated+=idtoken,
else
# Retrieve TRUST_DOMAIN from COLLECTOR_HOST and CCB if not already defined in condor_config (config attrs)
if out=$(get_trust_domain); then
export TRUST_DOMAIN="$out"
Expand Down

0 comments on commit 0c6f072

Please sign in to comment.