Skip to content

Conversation

@jackye1995
Copy link
Contributor

@jackye1995 jackye1995 commented Aug 23, 2025

Support use_fabric_endpoint config which will set the endpoint to fabric one instead of windows. This config cannot be set together with the endpoint config.

@jackye1995 jackye1995 requested a review from Xuanwo as a code owner August 23, 2025 04:36
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Aug 23, 2025
@Xuanwo
Copy link
Member

Xuanwo commented Aug 25, 2025

This'll be reviewed after upgrading reqsign.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Aug 26, 2025
@jackye1995 jackye1995 force-pushed the azure-missing-features branch from 9285657 to dc651c7 Compare August 26, 2025 19:41
The struct already has #[serde(default)] so individual fields don't need it

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jackye1995 jackye1995 changed the title feat(services/azblob): add missing Azure authentication features feat(services/azblob): support use_fabric_endpoint config Aug 26, 2025
@jackye1995
Copy link
Contributor Author

@Xuanwo thanks for taking a look. I removed all the auth-related chanages from the PR, will do it after reqsign upgrade. This PR now only contains the missing support of a use_fabric_endpoint config.

@jackye1995 jackye1995 closed this Aug 27, 2025
@jackye1995 jackye1995 reopened this Aug 27, 2025
@Xuanwo
Copy link
Member

Xuanwo commented Sep 2, 2025

@Xuanwo thanks for taking a look. I removed all the auth-related chanages from the PR, will do it after reqsign upgrade. This PR now only contains the missing support of a use_fabric_endpoint config.

I shares the same concern like #6531 (comment)

@jackye1995
Copy link
Contributor Author

I see, in that case I will close this one since this can be achieved by setting the endpoint directly. For S3 ones I think they are orthogonal, I replied more details there.

@jackye1995 jackye1995 closed this Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants