Skip to content
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.

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

Connection Timeout Error #215

Closed
wants to merge 14 commits into from
Closed

Connection Timeout Error #215

wants to merge 14 commits into from

Conversation

kibablu
Copy link
Contributor

@kibablu kibablu commented Apr 12, 2021

Add connection timeout error code on CONTRIBUTING.mb, the connection timeout
error is caused by poor internet connectivity as pip is set to default 15 sec.

When running this command with slow internet then pip throws a connection time error as pip is set to the default of 15 sec to download any dependency

python3 -m venv venv
./venv/bin/pip install -e '.[dev]'

upgrade black to the latest version

fixes #199

Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
upgrade flake8 to the latest version

fixes #199

Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
upgrade mypy to the latest version

fixes #199

Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
upgrade mypy-zope to the latest version

fixes #199

Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
fixes codestyle by running black on tests/ directory

fixes #199

Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
Add connection timeout error code on CONTRIBUTING.mb,the connection timeout
error is caused by poor internet connectivity as pip is set to default 15 sec.

Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
Copy link
Contributor

@callahad callahad left a comment

Choose a reason for hiding this comment

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

Hey, thank you so much! I've requested a few changes, after which this should be good to go

CONTRIBUTING.md Outdated Show resolved Hide resolved
changelog.d/215.doc Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
Copy link
Contributor

@callahad callahad left a comment

Choose a reason for hiding this comment

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

Great! Thank you so much 💯

@@ -1 +1 @@
Upgrade development dependencies fixes #199. Contributed by Omar Mohamed
Upgrade development dependencies fixes #199. Contributed by Omar Mohamed.
Copy link
Contributor

Choose a reason for hiding this comment

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

Oops! I noticed one final thing right before I hit merge 😆

Could you please rename this file to 215.misc instead of 214.misc, and maybe change the description to more closely match (e.g., "Add note about pip timeouts. Contributed by Omar Mohamed.")

Copy link
Contributor Author

@kibablu kibablu Apr 13, 2021

Choose a reason for hiding this comment

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

On it 😃
I just realized 214.misc was a PR on #199 issue and 215.doc is for this pip timeout error. Do you want it changed to be one changelog, or I change 215.doc to 215.misc

Copy link
Contributor

Choose a reason for hiding this comment

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

215.doc here would be fine :)

Copy link
Contributor Author

@kibablu kibablu Apr 13, 2021

Choose a reason for hiding this comment

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

I have made changes on the 215.doc changelog file with a more descriptive message on pip timeout, Thank you 😃

Copy link
Contributor

@callahad callahad left a comment

Choose a reason for hiding this comment

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

Works for me -- thank you so much 🎉

callahad pushed a commit that referenced this pull request Apr 13, 2021
Signed-off-by: Omar Mohamed <kibablu16@gmail.com>
@callahad
Copy link
Contributor

I manually merged this in 7a3d6f8 with a few small tweaks -- thank you again 😄

@callahad callahad closed this Apr 13, 2021
@kibablu
Copy link
Contributor Author

kibablu commented Apr 14, 2021

I manually merged this in 7a3d6f8 with a few small tweaks -- thank you again

Thank you so much 😃

awesome-michael added a commit to Awesome-Technologies/sygnal that referenced this pull request May 6, 2021
Sygnal 0.9.3 (2021-04-22)
=========================

Features
--------

- Prevent the push key from being rejected for temporary errors and oversized payloads, add TTL logging, and support `events_only` push data flag. ([\matrix-org#212](matrix-org#212))
- WebPush: add support for Urgency and Topic header ([\matrix-org#213](matrix-org#213))

Bugfixes
--------

- Fix a long-standing bug where invalid JSON would be accepted over the HTTP interfaces. ([\matrix-org#216](matrix-org#216))
- Limit the size of requests received from HTTP clients. ([\matrix-org#220](matrix-org#220))

Updates to the Docker image
---------------------------

- Remove manually added GeoTrust Root CA certificate from docker image as Apple is no longer using it. ([\matrix-org#208](matrix-org#208))

Improved Documentation
----------------------

- Make `CONTIBUTING.md` more explicit about how to get tests passing. ([\matrix-org#188](matrix-org#188))
- Update `CONTRIBUTING.md` to specify how to run code style and type checks with Tox, and add formatting to code block samples. ([\matrix-org#193](matrix-org#193))
- Document how to work around pip installation timeout errors. Contributed by Omar Mohamed. ([\matrix-org#215](matrix-org#215))

Internal Changes
----------------

- Update Tox to run in the installed version of Python (instead of specifying Python 3.7) and to consider specific paths and folders while running checks, instead of the whole repository (potentially including unwanted files and folders, e.g. the virtual environment). ([\matrix-org#193](matrix-org#193))
- Make development dependencies available as extras. Contributed by Hillery Shay. ([\matrix-org#194](matrix-org#194))
- Update `setup.py` to specify that a minimum version of Python greater or equal to 3.7 is required. Contributed by Tawanda Moyo. ([\matrix-org#207](matrix-org#207))
- Port CI checks to Github Actions. ([\matrix-org#210](matrix-org#210), [\matrix-org#219](matrix-org#219))
- Upgrade development dependencies. Contributed by Omar Mohamed ([\matrix-org#214](matrix-org#214))
- Set up `coverage.py` to run in tox environment, and add html reports ([\matrix-org#217](matrix-org#217))

Change-Id: I14ae821ff2a1562e91fd87ce25f73baaa0b9430b
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