-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
switch dependency for kerberos js module (ropnop/gorkb5 -> jcmturner/gokrb5) #4647
switch dependency for kerberos js module (ropnop/gorkb5 -> jcmturner/gokrb5) #4647
Conversation
@5amu thanks for this contribution! |
@tarunKoyalwar lgtm!
|
@5amu , thanks and everything looks good , just decided to do some refactoring to keep public api simple and use fastdialer instead of net.Conn ( added some cool helper utils to throw errors , create constructors and stuff)
^ will be converted to const client = new kerberos.Client(domain,controller)
const ticket = client.GetServiceTicket({Username: template.Username,Password: template.Password,Target: "roastme",SPN: "DC01/ldap"}) aka const client = new kerberos.Client(domain, controller)
const ticket = client.GetServiceTicket({
Username: template.Username,
Password: template.Password,
Target: "roastme",
SPN: "DC01/ldap"
}) this will not be a breaking change (we will keep existing ones) but this is recommended because new options/args can be added without breaking anything and this also adds code readability + intellisense if you use it any IDE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm ! tested it locally everything seems to working also added fastdialer + network policy .
thanks for pr @5amu , i just refactored kerberos struct / types to make it more easy to write js templates by leveraging autocomplete etc ( mostly adding a constructor throwing errors etc) . so if you have any nuclei templates written using this module you will need to update it after next release .
Proposed changes
Closes #4646
I updated the
KerberosClient
object to contain the real kerberos client fromgokrb5
:So that in file
sendtokdc.go
the struct can have the methodSendToKDC()
exposed toEnumerateUser
. The behavior stays the same, except now there is one less dependency to the project and (asgh.neting.cc/jcmturner
is used bygh.neting.cc/go-ldap/ldap/v3
under the hood).I tested the behavior with the template:
And here's the result
NB: The change reflected on
go.mod
andgo.sum
, please be careful.Checklist