Skip to content

Implement seek function for curl posts - #1728

Merged
Shelnutt2 merged 1 commit into
devfrom
sethshelnutt/ch879/need-to-implement-curlopt-seekfunction-for
Jul 20, 2020
Merged

Shelnutt2 merged 1 commit into
devfrom
sethshelnutt/ch879/need-to-implement-curlopt-seekfunction-for

Conversation

@Shelnutt2

Copy link
Copy Markdown
Contributor

This allows for redirects to be successful in POSTing data from curl over REST. A new BufferList::seek(offset, whence) function was added to facilitate this.

@Shelnutt2
Shelnutt2 requested review from ihnorton and joe-maley July 18, 2020 13:59

@joe-maley joe-maley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, a few style nitpicks

Comment thread tiledb/sm/buffer/buffer_list.h Outdated
Comment thread tiledb/sm/buffer/buffer_list.cc Outdated
Comment thread tiledb/sm/buffer/buffer_list.cc Outdated
@Shelnutt2
Shelnutt2 force-pushed the sethshelnutt/ch879/need-to-implement-curlopt-seekfunction-for branch from a69044d to 766c178 Compare July 18, 2020 19:02
This allows for redirects to be successful in POSTing data from curl
over REST. A new `BufferList::seek(offset, whence)` function was added
to facilitate this.
@Shelnutt2
Shelnutt2 force-pushed the sethshelnutt/ch879/need-to-implement-curlopt-seekfunction-for branch from 766c178 to 15a6fc5 Compare July 18, 2020 19:03
@Shelnutt2
Shelnutt2 merged commit 604f2fa into dev Jul 20, 2020
@Shelnutt2
Shelnutt2 deleted the sethshelnutt/ch879/need-to-implement-curlopt-seekfunction-for branch July 20, 2020 11:14
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.

2 participants