Skip to content

16.0 mig auth api key#451

Merged
OCA-git-bot merged 59 commits intoOCA:16.0from
benoit-bertholon:16.0-mig-auth_api_key
Dec 5, 2022
Merged

16.0 mig auth api key#451
OCA-git-bot merged 59 commits intoOCA:16.0from
benoit-bertholon:16.0-mig-auth_api_key

Conversation

@benoit-bertholon
Copy link
Copy Markdown
Contributor

No description provided.

lmignon and others added 30 commits November 29, 2022 22:34
Get rid of keychain dependency
Api key is now based on the new version of server env
Key belong to a specifiv database that can be requested based on the
domain name.

Just setup the db_filter with "%d^" to filter based on domain name
Add a migration script to create the auth.api.key records from keys defined into the cfg file
Provides a specific serction name builder to continue reading the key info from the same section as the one expected by the previous version
@StefanRijnhart StefanRijnhart mentioned this pull request Nov 30, 2022
@StefanRijnhart
Copy link
Copy Markdown
Member

Looking good now! Please squash your migration commits.

@benoit-bertholon
Copy link
Copy Markdown
Contributor Author

@StefanRijnhart squashed :)

Copy link
Copy Markdown
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

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

Thanks!

@benoit-bertholon
Copy link
Copy Markdown
Contributor Author

@StefanRijnhart sorry I pushed the wrong button ..

Copy link
Copy Markdown
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

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

No problem!

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@StefanRijnhart
Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-451-by-StefanRijnhart-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 21bac0e into OCA:16.0 Dec 5, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 6add8d6. Thanks a lot for contributing to OCA. ❤️

@yvaucher
Copy link
Copy Markdown
Member

yvaucher commented Jun 6, 2023

/ocabot migration auth_api_key

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jun 6, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 6, 2023
22 tasks
SiesslPhillip pushed a commit to grueneerde/OCA-server-auth that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-auth (14.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.