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

Add support for HTTP POST encryption ( using JWT ) #2926

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

Conversation

skyghost2210
Copy link

Added a new type of alert ( extend from HTTP POST ). This type will encrypt JSON alert using JWT ( JSON Web Tokens ).

Example of useage:

alert: jwt_post
http_post_jwt_key: "secret"
http_post_jwt_algorithm: "HS256"
http_post_url: "url_here"

@nsano-rururu
Copy link
Contributor

You need to add "PyJWT==1.7.1" To requirements.txt. Because running docker test uses requirements.txt instead of setup.py (see Docker-test file)

add PyJWT to requirements.txt
@skyghost2210
Copy link
Author

I have added "PyJWT==1.7.1" to requirements.txt. Can you check again? Thank you

@nsano-rururu
Copy link
Contributor

I am getting the following error.
You need to follow the fork / clone repository on GitHub to the original repository.

キャプチャ

Command execution example

git remote add upstream git://github.com/Yelp/elastalert.git
git fetch upstream

git merge upstream/master
git push origin master

@skyghost2210
Copy link
Author

I have resolved all the conflicts. Can you check again? Thank you

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