Skip to content
This repository was archived by the owner on Nov 7, 2022. It is now read-only.

Conversation

@jyap808
Copy link

@jyap808 jyap808 commented Aug 15, 2022

@dghubble
Copy link
Owner

Test please?

@jyap808
Copy link
Author

jyap808 commented Aug 15, 2022

Add a test case or test against the API?

@jyap808
Copy link
Author

jyap808 commented Aug 22, 2022

Added test case. Also confirming this was tested against the Twitter API and actually works.

@jyap808
Copy link
Author

jyap808 commented Aug 29, 2022

Mm, yeah on further testing with larger images (but within the 700kb documented limit) this doesn't work too well.

Errors:

  • 430 Request Header Fields Too Large
  • stream error: stream ID 1; INTERNAL_ERROR; received from peer

Doesn't look to be documented properly on the Twitter API.

The test case image works fine though.

@jyap808
Copy link
Author

jyap808 commented Aug 30, 2022

@dghubble OK it works now with the latest commit.

@jyap808
Copy link
Author

jyap808 commented Sep 14, 2022

Not sure if you’re still testing but this isn’t merged to Main branch.

@dghubble
Copy link
Owner

dghubble commented Nov 4, 2022

Thanks, but the project is being deprecated #207

@dghubble dghubble closed this Nov 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants