Skip to content

- Add Access token property and processing in Location object#774

Merged
AlejandroP merged 5 commits into
masterfrom
location_accesstoken
Dec 28, 2023
Merged

- Add Access token property and processing in Location object#774
AlejandroP merged 5 commits into
masterfrom
location_accesstoken

Conversation

@AlejandroP

Copy link
Copy Markdown
Contributor

Add AccessToken property to location object to access GAM protected services.

@genexusbot

Copy link
Copy Markdown
Collaborator
Cherry pick to beta success

@AlejandroP AlejandroP requested a review from iroqueta September 20, 2023 12:20
httpClient.addHeader("Content-Type", contentType);
}
}
if (this.location.getAuthenticationMethod() == 4 && this.location.getAccessToken() != null && ! this.location.getAccessToken().trim().isEmpty()) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why do you use this.location instead of only location?
The same with this.httpMethod

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Right, It's unnecessary, Removed.

- Fix test case when receiving unwrapped response.
@genexusbot

Copy link
Copy Markdown
Collaborator
Cherry pick to beta success

@genexusbot

Copy link
Copy Markdown
Collaborator
Cherry pick to beta success

@AlejandroP AlejandroP merged commit d1bdb18 into master Dec 28, 2023
@AlejandroP AlejandroP deleted the location_accesstoken branch December 28, 2023 19:03
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.

4 participants