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.

Sign up for GitHub

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

Jump to bottom

[14.0] [MIG] auth_api_key module #221

Merged
merged 33 commits into from
Dec 21, 2020
Merged

[14.0] [MIG] auth_api_key module #221

merged 33 commits into from
Dec 21, 2020

Conversation

chafique-delli
Copy link
Contributor

@chafique-delli chafique-delli commented Oct 12, 2020 •
edited
Loading

No description provided.

lmignon and others added 30 commits October 12, 2020 16:03
@lmignon @chafique-delli
New generic module du define REST services
16d1dd4
@lmignon @chafique-delli
auth_api_key: New module to provide authentication by api_key
6d34fd7
@lmignon @chafique-delli
auth_api_key: Remove useless code
d25b0f9
@RobinetDenisAcsone @chafique-delli
Update the auth_api_key for keychan 10.0.2 (renamed method)
6b96dfa
@RobinetDenisAcsone @chafique-delli
Fix the authentification method and show the user_id on the form
b179f78
@sebastienbeau @chafique-delli
refactor code in order to use generic code from base_rest instead of …
4720067 
…custom one from shopinvader
@sebastienbeau @chafique-delli
fix wrong name in selection box
d775ea3
@hparfr @chafique-delli
Since keychain 10.0.2.0.1 there is a default validation method
c606012
@qgroulard @chafique-delli
Keychain removed
6f84248 
Get rid of keychain dependency
@qgroulard @chafique-delli
Update Readme with more details
b9a36c1
@qgroulard @chafique-delli
Quick fix CONFIGURE.rst
3cd707e
@qgroulard @chafique-delli
Developmet status: Beta
9ba2ca0
@lmignon @chafique-delli
[FIX] auth_api_key: fix api and flake8
31ca88f
@sbidoul @chafique-delli
[MIG] server_auth from 10 to 12
cafb84a
@sbidoul @chafique-delli
auth_api_key: add description to model
b9765f8
@sbidoul @chafique-delli
[CHG] auth_api_key is LGPL
5ead097
@ap-wtioit @qgroulard @chafique-delli
Use consteq to compare keys
cd17ad5 
Co-Authored-By: qgroulard <43472442+qgroulard@users.noreply.github.com>
@OCA-git-bot @chafique-delli
[UPD] README.rst
a38f79c
@mourad-ehm @chafique-delli
[IMP] ADD possibility to save api keys in database by using server.en…
737db36 
…v.mixin obj
@mourad-ehm @chafique-delli
[FIX] login_name error
b0bbd84
@sebastienbeau @chafique-delli
[IMP] make auth_api_key compatible in multi-db
a76db1e 
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
@OCA-git-bot @chafique-delli
[UPD] README.rst
f634883
@lmignon @chafique-delli
[FIX] auth_api_key: Fix tests according to last changes and add views…
504985b 
… to manage auth api keys
@lmignon @chafique-delli
[IMP] auth_api_key: Ensures backward compatibility
c31347e 
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
@lmignon @chafique-delli
[FIX] auth_api_key: Cache must be cleared if the user_id is modified
137250b
@lmignon @chafique-delli
[IMP] auth_api_key: Improves key field description
0f84220
@OCA-git-bot @chafique-delli
[UPD] README.rst
be1678e
@chafique-delli
[IMP] auth_api_key: black, isort
3f2480d
@chafique-delli
[MIG] auth_api_key: Migration to 13.0
d9aa069
@oca-travis @chafique-delli
[UPD] Update auth_api_key.pot
0e27276
@chafique-delli chafique-delli mentioned this pull request Oct 13, 2020
Closed
17 tasks
@OCA-git-bot
Copy link
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). 🤖

@gurneyalex
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-221-by-gurneyalex-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 20, 2020
@OCA-git-bot
Merge PR #221 into 14.0
a089c3e 
Signed-off-by gurneyalex
@gurneyalex
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-221-by-gurneyalex-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 3, 2020
@OCA-git-bot
Merge PR #221 into 14.0
06d20a7 
Signed-off-by gurneyalex
@gurneyalex
Copy link
Member

@chafique-delli hello could you rebase your PR and recheck the precommit step, the merge by ocabot failed there, because of the website key in the manifest https://travis-ci.com/github/OCA/server-auth/jobs/453802850#L415

@sebastienbeau
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-221-by-sebastienbeau-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 9e2132d into OCA:14.0 Dec 21, 2020
@OCA-git-bot
Copy link
Contributor

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

@hparfr hparfr deleted the 14.0-mig-auth_api_key branch January 14, 2021 11:05
SiesslPhillip pushed a commit to grueneerde/OCA-server-auth that referenced this pull request Nov 20, 2024
@bt-admin
Merge pull request OCA#221 from OCA/14.0
d569052 
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
Reviewers

@gurneyalex gurneyalex gurneyalex approved these changes

@rvalyi rvalyi rvalyi approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.