Skip to content

Fix thumbnail token query parameter#1324

Merged
patrickarlt merged 3 commits into
Esri:mainfrom
ssylvia:main
Jul 1, 2026
Merged

Fix thumbnail token query parameter#1324
patrickarlt merged 3 commits into
Esri:mainfrom
ssylvia:main

Conversation

@ssylvia

@ssylvia ssylvia commented Jun 17, 2026

Copy link
Copy Markdown
Member

Fixes: #1314

Private item thumbnail URLs were appending the raw token as the query string, producing unauthorized requests. ArcGIS expects the token under the token query parameter.

  • Thumbnail URL decoration

    • Changed private thumbnail URLs from ?${token} to ?token=${token}.
    • Preserved public thumbnail behavior with no token appended.
  • Test coverage

    • Added direct decorateThumbnail() coverage for private and public thumbnail URLs.
    • Updated authenticated getItem() thumbnail expectations.
thumbnailUrl += `?token=${token}`;

@patrickarlt

Copy link
Copy Markdown
Contributor

LGTM! Thanks @ssylvia I'm going to merge.

@patrickarlt patrickarlt merged commit b4f180f into Esri:main Jul 1, 2026
2 of 3 checks passed
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.

decorateThumbnail token not applied correctly

3 participants