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

Investigate migration to symbol.exe for symbol publishing #14501

Open
2 tasks
hoyosjs opened this issue Feb 22, 2024 · 6 comments
Open
2 tasks

Investigate migration to symbol.exe for symbol publishing #14501

hoyosjs opened this issue Feb 22, 2024 · 6 comments

Comments

@hoyosjs
Copy link
Member

hoyosjs commented Feb 22, 2024

  • This issue is blocking
  • This issue is causing unreasonable pain

This is something that will become increasingly common and might become a blocker at some point in the future. Originally, the .NET team wrote a symbol uploader tool since the ecosystem didn't handle MACHO, ELF, Dwarf, and other files. Nowadays, through collaborations with the 1ES team there's a symbol uploader cli that supports these scenarios. This tool will be the supported solution going forward as it's actively maintained to follow best security practices. This tool supports more modern authentication workflows and is maintained by the same team that supports the backends for the

There are a few things to consider:

  • We'd have to explode nupkgs. The tool doesn't don't do this and has no practical reason to do so.
  • Explore CLR related keys - dac/dbi/clr key combo.
  • Full to portable PDB conversion is also not supported.

@epananth @mmitche

cc @mikem8361 @tommcdon @leculver

@hoyosjs
Copy link
Member Author

hoyosjs commented Feb 22, 2024

cc: @markwilkie

@markwilkie
Copy link
Member

Do you have any comments here @mikem8361 ?

@mikem8361
Copy link
Member

Juan has covered the issues here. We really can't support the current symbol uploader going forward especially for authentication,

@missymessa missymessa added this to the Tracking for other teams milestone Jun 13, 2024
@hoyosjs
Copy link
Member Author

hoyosjs commented Aug 22, 2024

@missymessa probably this shouldn't be in the "other teams" bucket. This will be a general infra requirement.

@missymessa
Copy link
Member

Followed up with Juan offline. Leaving this in "tracking for other teams".

@hoyosjs
Copy link
Member Author

hoyosjs commented Sep 11, 2024

#15055 is related.

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

No branches or pull requests

4 participants