Skip to content

[16.0] add base_external_system_ms_client_assertion #289

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
Open
NL66278 wants to merge 23 commits into
base: 16.0
Choose a base branch
from
Open

[16.0] add base_external_system_ms_client_assertion #289

NL66278 wants to merge 23 commits into from

Conversation

@NL66278
Copy link

@NL66278 NL66278 commented May 14, 2024

No description provided.

@NL66278 NL66278 force-pushed the 16.0-add-base_external_system_ms_client_assertion branch from e22688b to c458b29 Compare May 15, 2024 14:36
ntsirintanis
ntsirintanis reviewed May 21, 2024
View reviewed changes
@NL66278 NL66278 force-pushed the 16.0-add-base_external_system_ms_client_assertion branch from c9145b2 to ea7dd5f Compare May 22, 2024 08:06
@NL66278 NL66278 force-pushed the 16.0-add-base_external_system_ms_client_assertion branch 2 times, most recently from 1155011 to 990b53f Compare May 29, 2024 12:12
@NL66278 NL66278 changed the title [WIP][16.0] add base_external_system_ms_client_assertion [16.0] add base_external_system_ms_client_assertion Oct 7, 2024
@NL66278
Copy link
Author

NL66278 commented Oct 7, 2024

For some time now successfully running in production at large customer.

This was referenced Oct 22, 2024
Closed
Closed
SiesslPhillip pushed a commit to grueneerde/OCA-server-backend that referenced this pull request Nov 20, 2024
@bt-admin
Merge pull request OCA#289 from OCA/16.0
1f18821 
Syncing from upstream OCA/server-backend (16.0)
@github-actions
Copy link

github-actions bot commented Mar 23, 2025

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 23, 2025
@NL66278
Copy link
Author

NL66278 commented Mar 24, 2025

@thomaspaulb Could you add a no stale label here?

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 30, 2025
lfreeke
lfreeke approved these changes Jul 10, 2025
View reviewed changes
Copy link

@lfreeke lfreeke left a comment

Choose a reason for hiding this comment

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

👍

This have been working without issues for our customer

NL66278 and others added 15 commits September 3, 2025 13:01
@NL66278
[MIG] base_external_system: re-apply 10.0 changes
0c43ff0
@NL66278
[ADD] base_external_system_http: new module
e4a1704
@NL66278
[IMP] *_system_http: streamlined logging
1efbb01
@NL66278
[IMP] base_external_system_http: pre-commit stuff
226d670
@NL66278
[MIG] base_external_system_http: Migration to 16.0
50111c2
@NL66278
[ADD] base_external_system_oauth: new module
c01b121
@NL66278
[IMP] base_external_system_oauth: pre-commit stuff
f78b02f
@NL66278
[MIG] base_external_system_oath: migration to 16.0
cbc5e04
@NL66278
[IMP] *-system_oauth: provide tests
546a63f
@NL66278
[ADD] base_external_system_ms_client_assertion: new module
2a58db4
@NL66278
[TST] ..client_assertion: remove for now actual connection test
50e8276
@NL66278
[FIX] base_..._oauth: no need for basic type provider.
aa2b12c 
Basic authentication is no oauth at all and fully handled by base_external_system_http.
@NL66278
[IMP] base_..assertion: optionally use private_key value as is
992d43e 
In some installations, especially odoo.sh, it is not possible for the Odoo admin to put
files like the private key in the filesystem. In that case the private key can be pasted
into the private key field, and the value will be used and not interpreted as a filename.
@NL66278
[IMP] base_...assertion: add the xt5 header
6e594f0
@ntsirintanis @NL66278
[FIX] retrieve headers from kwargs
db73160
@NL66278 NL66278 force-pushed the 16.0-add-base_external_system_ms_client_assertion branch from 584630c to cbe2e26 Compare September 3, 2025 11:02
@NL66278 NL66278 mentioned this pull request Sep 3, 2025
Closed
@github-actions
Copy link

github-actions bot commented Jan 4, 2026

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 4, 2026
@thomaspaulb thomaspaulb added no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed stale PR/Issue without recent activity, it'll be soon closed automatically. labels Jan 5, 2026
@thomaspaulb
Copy link

thomaspaulb commented Jan 5, 2026

@NL66278 Sorry, this can't be merged in its current form. It's too many things at once, and covers 4 different modules.

Can we start by opening up a PR to base_external_system, with the refactors that are being done there? Add a description to the PR with checkboxes for each of the things that are added, and the source PR where it comes from (I can't find it, at least it was never merged)

@NL66278
Copy link
Author

NL66278 commented Jan 5, 2026

@thomaspaulb There was this one #281 that was also never merged. All of this is running in production, alt multiple clients, without any problems.

@NL66278
Copy link
Author

NL66278 commented Jan 27, 2026

@thomaspaulb OK, so here is the first PR for the base module. When that is merged, I will provide, one by one, PR's for the additional modules.

@NL66278
Copy link
Author

NL66278 commented Feb 9, 2026

@thomaspaulb Here is the actual link to the first PR, the one to make the base module extendable, #414

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@lfreeke lfreeke lfreeke approved these changes

@ntsirintanis ntsirintanis ntsirintanis approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved no stale Use this label to prevent the automated stale action from closing this PR/Issue.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@NL66278 @thomaspaulb @lfreeke @ntsirintanis @OCA-git-bot