adds headers and means to set them to Lbry apicall function #152

Merged
jessopb merged 1 commit from authHeaders into master 2019-06-03 19:04:43 +02:00
jessopb commented 2019-05-31 18:36:32 +02:00 (Migrated from github.com)
No description provided.
neb-b (Migrated from github.com) reviewed 2019-05-31 18:36:32 +02:00
kauffj (Migrated from github.com) requested changes 2019-05-31 23:20:09 +02:00
kauffj (Migrated from github.com) left a comment

Only strong request is to add a remove method.

Only strong request is to add a remove method.
kauffj (Migrated from github.com) commented 2019-05-31 23:18:05 +02:00

mildly in favor of calling this apiRequestHeaders

mildly in favor of calling this `apiRequestHeaders`
kauffj (Migrated from github.com) commented 2019-05-31 23:19:21 +02:00

There should presumably be a remove method as well.

I'm ambivalent between using add (add n) vs set (set 1).

There should presumably be a remove method as well. I'm ambivalent between using `add` (add `n`) vs `set` (set 1).
kauffj commented 2019-05-31 23:20:38 +02:00 (Migrated from github.com)

There's no need to submit this for a second review, just merge the changes.

There's no need to submit this for a second review, just merge the changes.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-redux#152
No description provided.