Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow specifying request headers in TilesetOptions #1057

Merged
merged 7 commits into from
Jan 22, 2025
Merged

Conversation

azrogers
Copy link
Contributor

@azrogers azrogers commented Jan 9, 2025

To support use cases like #738, this change allows specifying request headers in TilesetOptions that will be passed to the TilesetContentManager.

Copy link
Member

@kring kring left a comment

Choose a reason for hiding this comment

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

Looks great! Would you mind adding the same capability for RasterOverlays? It can be a separate PR if you want.

Cesium3DTilesSelection/src/Tileset.cpp Outdated Show resolved Hide resolved
@azrogers
Copy link
Contributor Author

Made #1067 for the task to implement the raster overlay equivalent, which I can get to soon.

@azrogers
Copy link
Contributor Author

Created the #1076 PR for headers for raster overlays.

@kring
Copy link
Member

kring commented Jan 22, 2025

Thanks @azrogers!

@kring kring merged commit 0dbb280 into main Jan 22, 2025
22 checks passed
@kring kring deleted the request-headers-tileset branch January 22, 2025 04:30
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.

2 participants