Skip to content

feat: ILP over HTTP (InfluxDB compatible) #50

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 68 commits into from
Jan 5, 2024
Merged

feat: ILP over HTTP (InfluxDB compatible) #50

merged 68 commits into from
Jan 5, 2024

Conversation

amunra
Copy link
Collaborator

@amunra amunra commented Dec 7, 2023 •
edited
Loading

Support for ILP over HTTP.

Features:

  • Basic auth
  • Bearer (token) auth
  • Auto-retry with timeout
  • Keepalive
  • Transactional flushes (i.e. restricted to a single table)

Additional minor Buffer API enhancements:

  • buffer.transactional()
  • buffer.row_count()

amunra and others added 30 commits December 7, 2023 16:33
@amunra
feat: ILP over HTTP (InfluxDB compatibility)
9225cd2
@amunra
simpler API for HTTP
b39b403
@amunra
C and C++ APIs
9d0991b
@amunra
tweaks and tests
8a614b4
@amunra
dependency updates and error code fix
d13cc63
@amunra
json parsing and more tests
006c951
@amunra
implemented (not tested yet) basic auth
a95a524
@amunra
headers tweak and manual testing of auth via http://httpbin.org/basic…
25840a0 
…-auth/foo/bar
@amunra
argument validation and exposing basic auth to C and C++
d092970
@amunra
error message parsing improvements in case of auth failure
35686ee
@amunra
Fixed a typo
6ac7d85
@amunra
auth test cases
1fe0edc
@amunra
proportional timeouts to request size
ed19ad9
@amunra
added precision=u url param and updated example
aa16866
@amunra
debris removal
f62bbfe
@amunra
user agent tweak
5b93efd
@amunra
test fixes
b747b0b
@amunra
bearer auth and ensuring http is optional
fbcf28d
@amunra
Token auth test
e7f8eb1
@amunra
timeout improvements and tests
825eb72
@amunra
token auth C and C++
f113897
@amunra
cargo fmt
4fff9f0
@amunra
tls proxy API for the win
bf09c02
@amunra
hopefully fixing tls_proxy
58e16ad
@amunra
HTTP TLS tests
eae1b91
@amunra
dropped mockito as test dependency and fully using own mock server al…
aaca75a 
…so for HTTP testing
@amunra
cargo fmt, cargo clippy
a210746
@amunra
more cargo fmt
f4414b2
@amunra
adding some timeouts for tests that are failing on windows
7340662
@amunra
hopefully fixing windows issues
7d735b3
@amunra amunra requested review from eugenels and jerrinot January 3, 2024 17:17
@amunra amunra marked this pull request as ready for review January 3, 2024 17:24
@amunra
Copy link
Collaborator Author

amunra commented Jan 3, 2024

For the related Python client changes, see questdb/py-questdb-client#59

@amunra amunra mentioned this pull request Jan 3, 2024
Merged
28 tasks
@amunra amunra changed the title feat: ILP over HTTP (InfluxDB compatibility) feat: ILP over HTTP (InfluxDB compatible) Jan 4, 2024
eugenels
eugenels reviewed Jan 4, 2024
View reviewed changes
@eugenels eugenels self-requested a review January 5, 2024 10:53
@amunra amunra merged commit aac4413 into main Jan 5, 2024
@amunra amunra deleted the ilp_http branch January 5, 2024 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eugenels eugenels eugenels approved these changes

@jerrinot jerrinot Awaiting requested review from jerrinot

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@amunra @eugenels