Skip to content

token downscoping for folders endpoints - #832

Merged
PJSimon merged 3 commits into
masterfrom
bug/folder-token-downscoping
Aug 20, 2020
Merged

PJSimon merged 3 commits into
masterfrom
bug/folder-token-downscoping

Conversation

@dtrodger

Copy link
Copy Markdown
Contributor
  • Adds folder API endpoint matching to determine a downscope token request resource attribute value

Comment thread src/main/java/com/box/sdk/BoxAPIConnection.java Outdated
Comment thread src/main/java/com/box/sdk/BoxAPIConnection.java Outdated
@PJSimon

PJSimon commented Aug 4, 2020

Copy link
Copy Markdown
Contributor

@dtrodger Build failing due to CheckStyle complaint...

@coveralls

coveralls commented Aug 12, 2020

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 2308

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.008%) to 65.983%

Totals Coverage Status
Change from base Build 2305: 0.008%
Covered Lines: 5759
Relevant Lines: 8728

💛 - Coveralls

@grey-lovelace

Copy link
Copy Markdown

@PJSimon is there anything else needed on this PR before merge? This issue is blocking my team from updating to the latest release, which we need for passing in the fields param to metadataQuery.

@PJSimon
PJSimon merged commit 066a8ad into master Aug 20, 2020
@PJSimon
PJSimon deleted the bug/folder-token-downscoping branch August 20, 2020 21:47
@PJSimon

PJSimon commented Aug 20, 2020

Copy link
Copy Markdown
Contributor

@grey-lovelace Sorry for the delay! I'll publish a release shortly! Thanks!

@PJSimon

PJSimon commented Aug 21, 2020

Copy link
Copy Markdown
Contributor

Hi @grey-lovelace,
The release has been deployed!
Thanks again!

@grey-lovelace

Copy link
Copy Markdown

@PJSimon thank you! Appreciate it.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants