Skip to content

Conversation

@QuantumTM
Copy link
Contributor

Domain encoding/decoding is handled by its basetype. This change forces
those basetypes to be inspected and loaded to the client cache avoiding
the issue where base types weren't encodable/decodable.

Domain encoding/decoding is handled by its basetype. This change forces those basetypes to be inspected and loaded to the client cache avoiding the issue where base types weren't encodable/decodable.
Copy link
Member

@elprans elprans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, good catch! Please add a test. Thanks!

Add test for domain basetype introspections and loading
@QuantumTM
Copy link
Contributor Author

test added, though I'm not sure its the best way to write it. Let me know if you think it should be done another way.

@elprans elprans merged commit cca4a2d into MagicStack:master Mar 25, 2022
elprans added a commit that referenced this pull request Jul 7, 2022
Changes ------- * Add support to use awaitable object in password function. (#889) (by @kwarunek in fb3b6bf for #889) * Support direct TLS connections (i.e. no STARTTLS) (#923) (by @jackwotherspoon in f2a937d for #923) Fixes ----- * Fix invalid `pyproject.toml` (#900) (by @Rongronggg9 in eddb649 for #900) * Add record_class parameter Pool.fetch and Pool.fetchrow (#896) (by @baltitenger in 2519cf3 for #896) * Domain basetypes are introspected (#886) (#887) (by @QuantumTM in cca4a2d for #886) * Properly handle exceptions raised while handling server auth messages (#862) (by @elprans in bd19262 for #862)
elprans added a commit that referenced this pull request Jul 7, 2022
Changes ------- * Add support to use awaitable object in password function. (#889) (by @kwarunek in fb3b6bf for #889) * Support direct TLS connections (i.e. no STARTTLS) (#923) (by @jackwotherspoon in f2a937d for #923) Fixes ----- * Fix invalid `pyproject.toml` (#900) (by @Rongronggg9 in eddb649 for #900) * Add record_class parameter Pool.fetch and Pool.fetchrow (#896) (by @baltitenger in 2519cf3 for #896) * Domain basetypes are introspected (#886) (#887) (by @QuantumTM in cca4a2d for #886) * Properly handle exceptions raised while handling server auth messages (#862) (by @elprans in bd19262 for #862)
elprans added a commit that referenced this pull request Jul 7, 2022
Changes ------- * Add support to use awaitable object in password function. (#889) (by @kwarunek in fb3b6bf for #889) * Support direct TLS connections (i.e. no STARTTLS) (#923) (by @jackwotherspoon in f2a937d for #923) Fixes ----- * Fix invalid `pyproject.toml` (#900) (by @Rongronggg9 in eddb649 for #900) * Add record_class parameter Pool.fetch and Pool.fetchrow (#896) (by @baltitenger in 2519cf3 for #896) * Domain basetypes are introspected (#886) (#887) (by @QuantumTM in cca4a2d for #886) * Properly handle exceptions raised while handling server auth messages (#862) (by @elprans in bd19262 for #862)
@elprans elprans mentioned this pull request Jul 7, 2022
rohitsanj pushed a commit to noteable-io/asyncpg-crdb-noteable that referenced this pull request May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants