Skip to content

User/manishkg/remove map token#257

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

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

Conversation

@srijan55
Copy link
Copy Markdown
Contributor

@srijan55 srijan55 commented Jul 31, 2025

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