JanssenProject / JanssenProject/jans

feat(config-api): Blacklist some urls from webhook for security reason

Open
#8,574 13 comments 0 reactions 1 assignee Claimed by @433B View on GitHub
kind-feature
Dominant language
Java
Stars
648
Forks
174
Avg merge
1d 18h
Merged PRs (30d)
110

Description

-------------------

### Description

Currently there is no security in webhook feature.
The following should be check:

1. Perform URL validation(url should be valid url and resolvable).
1. Ensure URL starts with "https://", disallow "file://" and other non-HTTPS schemes.
1. Block typical local IPs: 127.0.x, 192.168.x, 172.x.
1. Prohibit "localhost" and "http://"
1. Require a specific response header for POST requests, unique to the customer.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.