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

Development #141

Merged
merged 15 commits into from
Oct 23, 2023
Merged

Development #141

merged 15 commits into from
Oct 23, 2023

Conversation

LorenzoJokhan
Copy link
Contributor

Description

Please include

  • a summary of the changes
  • relevant motivation and context
  • a list of any dependencies that are required for this change

Issue reference

Fixes # (issue)

Type of change

Is it a new feature, bug fix, code improvement, etc.
If it is a breaking change what needs to be done to fix that

Documentation

Is the documentation updated, maybe a link

Tests

(How) has the change been tested

Branch

If the branch to merge to is not development

stijnvandervegt and others added 15 commits January 27, 2022 14:52
This routes /url/authenticate requests from the Cisco Umbrella CIDRs to a different page. This prevents Cisco from 'logging in' and preventing the user from logging in.
You can now add `blockCidrs` to the client config.

Example:
```
{
	"blockCidrs": [
		"1.2.3.4/16",
		"4.4.4.4/16"
	]
}
```

When the `blockCidrs` key does not exist in the client config, the block cidrs default to the Cisco umbrella CIDRs.

Note: This means that if you want to add CIDRs to the block CIDRs, and you want to keep the Cisco Umbrello CIDRs, you have to add those to the client config as well.
@LorenzoJokhan LorenzoJokhan merged commit 9f61bc4 into master Oct 23, 2023
1 check passed
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

3 participants