Skip to content

fix: Removed url pattern validation for google urls in external account credential configurations#1150

Merged
aeitzman merged 2 commits intogoogleapis:mainfrom
aeitzman:remove-url-validation
Feb 11, 2023
Merged

fix: Removed url pattern validation for google urls in external account credential configurations#1150
aeitzman merged 2 commits intogoogleapis:mainfrom
aeitzman:remove-url-validation

Conversation

@aeitzman
Copy link
Contributor

@aeitzman aeitzman commented Feb 7, 2023

Removed pattern checking for external accounts to support TPC urls. Also added readme change to explain risk to users.

@aeitzman aeitzman requested a review from a team as a code owner February 7, 2023 22:03
@aeitzman aeitzman requested a review from a team February 7, 2023 22:03
@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label Feb 7, 2023
Copy link

@TimurSadykov TimurSadykov left a comment

Choose a reason for hiding this comment

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

LGTM

@aeitzman aeitzman merged commit 35495b1 into googleapis:main Feb 11, 2023
gcf-merge-on-green bot pushed a commit that referenced this pull request Feb 15, 2023
🤖 I have created a release *beep* *boop*
---


## [1.16.0](https://togithub.com/googleapis/google-auth-library-java/compare/v1.15.0...v1.16.0) (2023-02-15)


### Features

* Add PKCE to 3LO exchange. ([#1146](https://togithub.com/googleapis/google-auth-library-java/issues/1146)) ([5bf606b](https://togithub.com/googleapis/google-auth-library-java/commit/5bf606bb8f6d863b44e87587eebf51eaeea4a0ae))


### Bug Fixes

* Create and reuse self signed jwt creds for better performance ([#1154](https://togithub.com/googleapis/google-auth-library-java/issues/1154)) ([eaaa8e8](https://togithub.com/googleapis/google-auth-library-java/commit/eaaa8e89cf69d1e0d581443121f315854d52c75f))
* Java doc for DefaultPKCEProvider.java ([#1148](https://togithub.com/googleapis/google-auth-library-java/issues/1148)) ([154c127](https://togithub.com/googleapis/google-auth-library-java/commit/154c1279b3ec96cc34a3225e5e78800ccdda927c))
* Removed url pattern validation for google urls in external account credential configurations ([#1150](https://togithub.com/googleapis/google-auth-library-java/issues/1150)) ([35495b1](https://togithub.com/googleapis/google-auth-library-java/commit/35495b1207ffe11712ee996d3e305449752fb87c))


### Documentation

* Clarified Maven artifact for HTTP-based clients ([#1136](https://togithub.com/googleapis/google-auth-library-java/issues/1136)) ([b49fc13](https://togithub.com/googleapis/google-auth-library-java/commit/b49fc13b10d0e326c7296e2aad7a50ea03e774f5))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments