fix: packet referral decoding

[james-d-elliott]

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.

[cpuschma]

LGTM! Thank you for your PR! 🥰

[james-d-elliott]

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]

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]

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.