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

Metadata: Allow authenticated mongodb connections; Fix #6605 #6758

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

maxnoe
Copy link
Contributor

@maxnoe maxnoe commented May 6, 2024

Follow up to #6606 following the contributors guide

@maxnoe maxnoe force-pushed the feature-6605-mongo_authentication branch 3 times, most recently from b78aaf7 to b467dde Compare May 7, 2024 11:30
lib/rucio/core/did_meta_plugins/mongo_meta.py Outdated Show resolved Hide resolved
tests/test_did_meta_plugins.py Outdated Show resolved Hide resolved
@maxnoe maxnoe force-pushed the feature-6605-mongo_authentication branch 2 times, most recently from 9d2a43f to 051ef0a Compare May 29, 2024 11:42
Copy link
Contributor

@rdimaio rdimaio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ruff failures are due to an unrelated issue that was fixed in a separate PR (they will go away if you rebase, but there's no need to do that since there's no merge conflicts)

AUTHORS.rst Show resolved Hide resolved
lib/rucio/core/did_meta_plugins/mongo_meta.py Show resolved Hide resolved
@maxnoe maxnoe force-pushed the feature-6605-mongo_authentication branch from 051ef0a to 17e5afe Compare June 3, 2024 16:02
@maxnoe maxnoe force-pushed the feature-6605-mongo_authentication branch from 17e5afe to 62e8e4b Compare June 4, 2024 09:15
@bari12 bari12 merged commit a98ce1a into rucio:master Jun 4, 2024
27 checks passed
@maxnoe maxnoe deleted the feature-6605-mongo_authentication branch June 4, 2024 11:55
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.

None yet

3 participants