SharpHoundCommon icon indicating copy to clipboard operation
SharpHoundCommon copied to clipboard

fix TGTDelegationEnabled

Open JonasBK opened this issue 1 year ago • 0 comments

Description

Do not use CrossOrganizationNoTGTDelegation to calculate TGTDelegationEnabled.

Motivation and Context

CrossOrganizationNoTGTDelegation being false does not enable TGT delegation. That is what my testing shows. It seems like this flag became meaningless after TGT delegation was disabled by default in 2019.

How Has This Been Tested?

Locally in my lab.

Screenshots (if appropriate):

Types of changes

  • [ ] Chore (a change that does not modify the application functionality)
  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [ ] Documentation updates are needed, and have been made accordingly.
  • [ ] I have added and/or updated tests to cover my changes.
  • [ ] All new and existing tests passed.
  • [ ] My changes include a database migration.

JonasBK avatar Oct 02 '24 14:10 JonasBK