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

Application layer checks (optional) in the network checker #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ralienpp
Copy link

@ralienpp ralienpp commented Oct 3, 2017

Added ability to check if the server replies with a particular message, to ensure it is really alive.

@JeanFred
Copy link
Member

JeanFred commented Oct 4, 2017

That looks great @ralienpp. I’ll get down to review it further ASAP.

I can already see it’s missing migrations, but I can add that :)

@ralienpp
Copy link
Author

ralienpp commented Oct 4, 2017

I'd be glad to add them myself, but my knowledge of Django is superficial and rusty. If you can write a step-by-step guide, I will be able to follow it; otherwise I am not confident I won't mess up the existing ORM-related logic.

Otherwise, I'm happy to write the purely network-related part.

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.

None yet

2 participants