Skip to content

User/manishkg/remove map token#257

Merged
srijan55 merged 3 commits into
mainfrom
user/manishkg/removeMapToken
Jul 31, 2025
Merged

User/manishkg/remove map token#257
srijan55 merged 3 commits into
mainfrom
user/manishkg/removeMapToken

Conversation

@srijan55

@srijan55 srijan55 commented Jul 31, 2025

Copy link
Copy Markdown
Contributor

Description

Removed the unused endpoint /map/token

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • [*] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Validated locally. Ran unit and integration tests.

Checklist:

Please delete options that are not relevant.

  • I have performed a self-review
  • Changelog has been updated
  • Documentation has been updated
  • Unit tests pass locally (./scripts/test)
  • Code is linted and styled (./scripts/format)

@ghidalgo3

Copy link
Copy Markdown
Collaborator

How do users get map tokens now? Is the subscription key served in the frontend?

microsoft/PlanetaryComputerDataCatalog#507

@srijan55

Copy link
Copy Markdown
Contributor Author

How do users get map tokens now? Is the subscription key served in the frontend?

microsoft/PlanetaryComputerDataCatalog#507

Yes. The subscription key is served via github secrets and part of frontend deploy now.

@ghidalgo3

Copy link
Copy Markdown
Collaborator

This ended up being the problem we were facing: docker/compose#13013

@srijan55
srijan55 merged commit 7b5d499 into main Jul 31, 2025
6 checks passed
@srijan55
srijan55 deleted the user/manishkg/removeMapToken branch July 31, 2025 18:03
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.

3 participants