Skip to content

update to new zig async io#100

Closed
philipmv wants to merge 1 commit intokarlseguin:devfrom
philipmv:dev
Closed

update to new zig async io#100
philipmv wants to merge 1 commit intokarlseguin:devfrom
philipmv:dev

Conversation

@philipmv
Copy link
Copy Markdown
Contributor

@philipmv philipmv commented Nov 12, 2025

This is just as an experiment may be it should be in a separate branch. Unfortunately ssl connection is removed.

@karlseguin
Copy link
Copy Markdown
Owner

I'm not really following along...is TLS not currently working in Zig-dev, or just not ported in this PR yet?

@philipmv
Copy link
Copy Markdown
Contributor Author

No, it's working. I just haven't looked into it and didn't port it. This PR is only useful if you are interested in using pg.zig with zig's new async io. It's very early, but someone may still be interested.

@lalinsky
Copy link
Copy Markdown

I was just looking at using pg.zig with my zio library, and though I could use this as the base, but it's very incomplete:

  • still uses threads, especially for the reconnector
  • timeouts need to be restored
  • openssl will be a bigger issue, will either need to migrate to std.Io.Reader and use std.crypto.tls, or use openssl BIO APIs.

I'll give it a try to refactor the lib to use standard reader/writer, which would allow it to use std.crypto, that might be the easiest option.

@zigster64
Copy link
Copy Markdown
Contributor

@philipmv - got an attempt number 2 on this one over here #107, which is really close to your original one passing Io around, and a few other changes to bring it up to 0.16 latest

0.16 has probably changed a bit since you posted this one - be good if you could have a play as well, and review this additional PR

Builds and passes tests on 0.16-2565

@philipmv philipmv deleted the branch karlseguin:dev April 16, 2026 12:19
@philipmv philipmv closed this Apr 16, 2026
@philipmv philipmv deleted the dev branch April 16, 2026 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants