go-ldap / ldap

Basic LDAP v3 functionality for the GO programming language.
Other
2.25k stars 355 forks source link

fix: packet referral decoding #413

Closed james-d-elliott closed 1 year ago

james-d-elliott commented 1 year ago

This adjusts the packet referral decoding behavior to more appropriately handle edge cases and allows to more easily obtain the underlying packet in the event of a decoding error. Instead of matching ASN.1 BER packets on the tag for referrals we match on the class and type. This is particularly important in regards to OpenLDAP which returns a ASN.1 BER Generalized Time tag for referrals.

While I think this is a bug on the OpenLDAP side it's also perfectly reasonable in my opinion to handle this here as it's relatively easy to do so.

james-d-elliott commented 1 year ago

By the way I know a lot of people are busy lately and 2 weeks is not really that long, and it's by no means a complaint just curious. In the future is there a way to get your attention that isn't irritating and that you'd welcome?

cpuschma commented 1 year ago

Sorry for the silence in between, I understand that this can be annoying for you who lovingly creates a PR.

I can't speak for the other maintainers, but I think one way would be you mention @go-ldap/committers. Alternatively if a topic is urgent or something has gone unnoticed for too long, my email box is open cp at lumen.sh.

james-d-elliott commented 1 year ago

Sorry for the silence in between, I understand that this can be annoying for you who lovingly creates a PR.

I can't speak for the other maintainers, but I think one way would be you mention @go-ldap/committers. Alternatively if a topic is urgent or something has gone unnoticed for too long, my email box is open cp at lumen.sh.

Yeah it's no worries at all, I really want to stress that am usually very direct / blunt to a fault (what you see is what you get). Life can get busy, especially in the early months which I totally understand. I just wondered if there was a preferred way that wasn't going to bother you guys.