OctopusDeploy / Halibut

| Public | A secure communication stack for .NET using JSON-RPC over SSL.

Other

12 stars 44 forks source link

Mark RemoveTrust() as Obsolete #480

Closed LukeButters closed 1 year ago

LukeButters commented 1 year ago

Background

[SC-58250]

We don't use this method and it is not as safe as just always using "trust only".

Related to https://github.com/OctopusDeploy/Issues/issues/8266

How to review this PR

Quality :heavy_check_mark:

Pre-requisites

[ ] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[ ] I have considered informing or consulting the right people, according to the ownership map.
[ ] I have considered appropriate testing for my change.

shortcut-integration[bot] commented 1 year ago

This pull request has been linked to Shortcut Story #58250: Mark HalibutRuntime.RemoveTrust as Obsolete.