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(Resources): hide private key from the logs #7835

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

aldbr
Copy link
Contributor

@aldbr aldbr commented Oct 15, 2024

Closes #7834
BEGINRELEASENOTES
*Resources
FIX: hide private keys from the logs
ENDRELEASENOTES

@aldbr aldbr requested a review from sfayer October 15, 2024 06:11
@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Oct 15, 2024
@fstagni fstagni merged commit ab3e80b into DIRACGrid:rel-v8r0 Oct 16, 2024
21 of 26 checks passed
@DIRACGridBot DIRACGridBot added sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention labels Oct 16, 2024
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/11361802486

Failed:

  • integration
    cherry-pick ab3e80b into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-ab3e80bb5-integration
    git cherry-pick -x -m 1 ab3e80bb5
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #7835 fix(Resources): hide private key from the logs' --author='aldbr <[email protected]>'
    git push -u origin cherry-pick-2-ab3e80bb5-integration
    
    # If you have the GitHub CLI installed the PR can be made with
    gh pr create \
         --label 'sweep:from rel-v8r0' \
         --base integration \
         --repo DIRACGrid/DIRAC \
         --title '[sweep:integration] fix(Resources): hide private key from the logs' \
         --body 'Sweep #7835 `fix(Resources): hide private key from the logs` to `integration`.
    
    Adding original author @aldbr as watcher.
    
    BEGINRELEASENOTES
    *Resources
    FIX: hide private keys from the logs
    ENDRELEASENOTES
    Closes #7838'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants