Specify file parameter for API.update_profile_banner explicitly

Improve API.update_profile_image file uploading

Use files parameter of requests Session.request
Allow Twitter's API to respond with "Image validation failed with errors: Image size must be <= 2.0 MiB." and "media type unrecognized." errors instead of checking for file size and type prior to upload.

Rename API.update_profile_image file_ parameter to file

Remove unnecessary allowed_param argument for API.create_media_metadata

Remove unnecessary allowed_param arguments for media upload methods

API.simple_upload, API.chunked_upload_init, API.chunked_upload_append, API.chunked_upload_finalize

Remove payload_type and parser arguments for API.chunked_upload_append

Improve comments in API.chunked_upload

Remove image types constant

Only used once in API._pack_image

Add media_category parameter to API.chunked_upload

Replace is_direct_message parameter
Remove API._get_media_category

This allows for the media_category parameter to be more clearly set and set with values outside of those that were previously hardcoded, e.g. amplify_video.

Add media_category parameter to API.simple_upload

Use initial file pointer location in API.chunked_upload

Rather than resetting to the beginning of the passed file

Replace file size check in API.media_upload with chunked parameter

The simple upload endpoint seems to accept media file sizes much larger than documented, as long as the size after Twitter's API processes the media is below 5 MiB (e.g. through compression, stripping whitespace, etc.). This makes any file size checks prior to uploading to the API potentially overly restrictive.

Regardless of this, MAX_UPLOAD_SIZE_STANDARD was already lower than necessary anyway, as the limit appears to be 5 MiB rather than 5 MB, as the error message when using chunked upload for a file larger than 5 MiB, "File size exceeds 5242880 bytes.", indicates.

The simple upload endpoint seems to consistently respond with a 413 Payload Too Large HTTP response status code when the uploaded file is too large, even after being processed, which should make it fairly apparent that the file is too large.

Most GIFs should be under 5 MiB, but for larger GIFs that the simple upload endpoint will error for, using the chunked parameter will be necessary. Adding this parameter also allows chunked upload of images less than 5 MiB as desired.

Remove file type check in API.media_upload

The error message Twitter's API responds with when the file type is unsupported, "media type unrecognized.", is fairly self-explanatory.
Allowing Twitter's API to handle the media type check and avoiding hardcoding supported types also removes the need to update them in the future if they change.

Handle upload chunk size in bytes

Remove default and max chunk size constants

Only used once each in API.chunked_upload

Base minimum chunk size for uploads on actual file size

MIN_CHUNKSIZE was too small for possible video sizes regardless

Remove max file size checks for chunked uploads

The error messages Twitter's API responds with when the file size is too large, "File size exceeds 15728640 bytes." for GIFs and "File size exceeds 536870912 bytes." for videos, are fairly self-explanatory.
This check was incorrect for videos anyway, which seem to have a size restriction of 512 MiB, not 15 MB.
https://developer.twitter.com/en/docs/twitter-api/v1/media/upload-media/uploading-media/media-best-practices
https://twittercommunity.com/t/conflicting-documented-size-restrictions-for-video-uploading-via-api/148201
Allowing Twitter's API to handle the size restrictions and avoiding hardcoding them also removes the need to update them in the future if they change.

Fix logic in determining number of segments in API.chunked_upload

Optimize determination of number of segments in API.chunked_upload

Reuse previously determined file size in API.chunked_upload

Fixes file size determination with file parameter

Simplify API.chunked_upload

Improve variable names in API.chunked_upload

Handle additional_owners parameter in API.chunked_upload_init

Rename API.simple_upload and API.chunked_upload f parameters to file

Rename variable from f to file in API.media_upload

Specify as parameter explicitly

Specify API.chunked_upload parameters explicitly

Fix API.chunked_upload file object parameter name

Remove urllib.parse.urlencode import

Refactor API.chunked_upload

Add API.chunked_upload_init, API.chunked_upload_append, API.chunked_upload_finalize
Remove API._chunk_media
Use requests Session.request data and files parameters

Rename API.image_upload to API.simple_upload

Rename API.upload_chunked to API.chunked_upload

Update cassettes for API.media_upload tests with GIF and PNG

Use files parameter of requests Session.request for API.image_upload

Fix file parameter size determination in API.media_upload

Improve clarity of API.media_upload

Replace str.format usage with f-strings

Replace old C-style string formatting/interpolation with f-strings

Merge branch 'master' into video-upload

Set COVERALLS_SERVICE_NAME to github in GitHub Actions Test workflow

https://github.com/coveralls-clients/coveralls-python/issues/240
https://github.com/coveralls-clients/coveralls-python/pull/249

Replace os.error with OSError

Remove API._remove_list_members

Remove API._add_list_members

Remove API._lookup_friendships

Allow sending Call-To-Action buttons with Direct Messages

In API.send_direct_message
Resolves #1311

Improve clarity of and optimize API.send_direct_message

Allow sending Quick Reply Options with Direct Messages

Replace API.send_direct_message quick_reply_type parameter with quick_reply_options
Resolves #1280

Merge pull request #1487 from tweepy/python-3.5-support-removal

Drop support for Python 3.5

Improve formatting in Status and User methods

Merge pull request #1282 from okomestudio/ts/lists_ownerships

Implement lists/ownerships

Update cassette for API.lists_ownerships test

Merge branch 'master' into ts/lists_ownerships

Update tweepy/models.py

Co-authored-by: Harmon <Harmon758@gmail.com>

Update tweepy/api.py

Co-authored-by: Harmon <Harmon758@gmail.com>

Update docs/api.rst

Co-authored-by: Harmon <Harmon758@gmail.com>

Update tweepy/api.py

Co-authored-by: Harmon <Harmon758@gmail.com>

Update docs/api.rst

Co-authored-by: Harmon <Harmon758@gmail.com>

Update docs/api.rst

Co-authored-by: Harmon <Harmon758@gmail.com>

Remove API._send_direct_message

Remove unnecessary allowed_param argument in API._send_direct_message

Merge branch 'master' into python-3.5-support-removal

Reduce extra sleep time for rate limit handling in APIMethod.execute

Resolves #1049

Improve clarity of rate limit handling in APIMethod.execute

Also improve comment capitalization consistency and remove unnecessary comments

Optimize HTTP status code checks and retry handling in APIMethod.execute

Also improve formatting

Improve APIMethod.execute check for 420 and 429 HTTP status codes

Also improve formatting and consistency in comment capitalization

Remove explicit handling of requests.ReadTimeout in Stream

requests.ReadTimeout subclasses requests.Timeout, so this shouldn't be necessary

Handle requests.ReadTimeout in Stream

Resolves #750

Simplify requests imports in streaming.py

Rename StreamListener.keep_alive to StreamListener.on_keep_alive

To be consistent with other StreamListener methods

Improve variable name in Stream

Remove Stream._data method

Remove unused variable in Stream

Remove parse_a_href and parse_html_value functions

Replace convert_to_utf8_str function with str cast

Optimize list_to_csv function

Replace parse_datetime function with email.utils.parsedate_to_datetime

This causes the datetime objects for created_at attributes to be aware

Update copyright years to include 2021

Add and use mixins for Status and User

Merge pull request #1306 from mef51/patch-1

make User object hashable to allow use with sets

hash == id

Co-authored-by: Harmon <Harmon758@gmail.com>

Merge branch 'master' into patch-1

use id as hash since its an integer

Co-authored-by: Harmon <Harmon758@gmail.com>

Improve formatting in streaming.py

Check consumer key and secret type when initializing OAuthHandler

Resolves #1489

Remove unnecessary ASCII encoding of credentials in OAuthHandler

Remove Status.__ne__ and User.__ne__

Improve exception handling in Stream

Remove StreamListener.on_event

Simplify StreamListener.on_data

Merge pull request #1484 from dplefevre/doc_fix

Doc fix

Improve formatting for documentation for include_rts parameter

In documentation for API.user_timeline

Update cassette for user_timeline test

Update user_timeline test

Remove Stream.api

Simplify Stream.disconnect

Improve inline comment formatting in Stream._read_loop

Add Python Versions badge to README

Remove unnecessary URL parameter in Documentation Status badge in README

Improve PyPI Version badge in README

Add links to documentation and Twitter API documentation to README

Also improve Discord server invite link text

Improve Discord badge in README