Skip to content

Ticketer: Added extra-pac implementation#1545

Merged
0xdeaddood merged 8 commits intomasterfrom
ticketer_extra_pac
May 5, 2023
Merged

Ticketer: Added extra-pac implementation#1545
0xdeaddood merged 8 commits intomasterfrom
ticketer_extra_pac

Conversation

@0xdeaddood
Copy link
Collaborator

Extra-PAC implementation by @Dramelac. Additions detailed in #1391.

This PR adds a new SID structure [MS-DTYP] 2.4.2.2 SID. This is a packet representation of the SID type for use by block protocols.
Removed LDAP_SID structure in the example.

Example of use:
ticketer.py -aesKey 78738d... -domain-sid S-1-5-21-22... -domain contoso.com -extra-pac -user-id 1108 username

@0xdeaddood 0xdeaddood merged commit c309363 into master May 5, 2023
@Dramelac
Copy link
Contributor

Dramelac commented May 5, 2023

Are you sure @0xdeaddood about 0e975ee ? As I said in the original PR I put these pac by default because of the new Microsoft policy that need them every time and to avoid mistake in the new 'default' behavior.
Maybe add a -legacy flag to use the old pac structure would be a better UX approach ?

@0xdeaddood
Copy link
Collaborator Author

@Dramelac I think your approach is better. I opened #1546 with the changes (if you could review it 🙏). The default ticket structure will include the new PAC structures. If the old structure is needed, you can use the -old-pac option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants