-
Notifications
You must be signed in to change notification settings - Fork 50
docs: [JS] add IAS App-to-App #2335
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
base: main
Are you sure you want to change the base?
Conversation
3ef293c to
7233daa
Compare
372317e to
e3bdb6a
Compare
* origin/main: Bump prettier-plugin-java from 2.7.7 to 2.8.0 (#2349) Java 5.25.0: Add docu for TP loader (#2290) Bump dependabot/fetch-metadata from 2.4.0 to 2.5.0 (#2348) Bump @typescript-eslint/eslint-plugin from 8.51.0 to 8.52.0 (#2346) Bump prettier from 3.5.3 to 3.7.4 (#2324) Update SAP Cloud SDK for JavaScript API documentation update release notes (#2343) Update SAP Cloud SDK for JavaScript API documentation Bump globals from 16.5.0 to 17.0.0 (#2342) Bump @typescript-eslint/eslint-plugin from 8.50.1 to 8.51.0 (#2341) Bump @typescript-eslint/parser from 8.50.1 to 8.51.0 (#2340) Bump @typescript-eslint/eslint-plugin from 8.50.0 to 8.50.1 (#2339) Bump @typescript-eslint/parser from 8.50.0 to 8.50.1 (#2338) chore: Update JavaDocs for release 5.25.0 (#2337) Java: Add release notes for release 5.25.0 (#2336)
4a949fc to
ff463a9
Compare
ff463a9 to
e95931a
Compare
1f199e2 to
d32e6b0
Compare
Co-authored-by: KavithaSiva <kavitha.sivakumar@sap.com>
KavithaSiva
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few more comments. Like I stated below, I cross-checked Java's implementation and confirmed that the subdomain replacement happens for both client credentials and jwt bearer flows if the subdomain information can be determined.
Co-authored-by: KavithaSiva <kavitha.sivakumar@sap.com>
Co-authored-by: KavithaSiva <kavitha.sivakumar@sap.com>
|
@KavithaSiva Good catch! I'll write a quick PR for cloud-sdk-js to change that. |
What Has Changed?
Explain what you are changing and why, if it isn't obvious from the diff.