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

Provide IP Address in Duo Request #18662

Closed
gw-uber opened this issue Jan 11, 2023 · 2 comments
Closed

Provide IP Address in Duo Request #18662

gw-uber opened this issue Jan 11, 2023 · 2 comments

Comments

@gw-uber
Copy link

gw-uber commented Jan 11, 2023

We are looking for Vault to provide the IP address of the request coming from Vault to a Duo challenge after successful LDAP authentication.

We are trying to implement IP address restrictions for our instance of Vault. However, when a Duo challenge request is made, there is no IP address (IPv4 or v6) passed with that request (see screenshot from Duo audit logs, Vault sends 0.0.0.0)
no-ip-address. We are requesting that Vault send it's host system's IP address (ex: our AWS cluster where Vault is deployed) with any Duo request so that it can be recorded in Duo's logs and if not on the Duo whitelist, blocked.

This is a p0 blocker for us as we need Vault to send the IP address of the host/requesting system to Duo so our Duo api can record the incoming IP address from these access requests, log them, and filter those that are not in the range that we have whitelisted.

The desired outcome for this FR is that when a Duo mfa challenge happens, the corresponding request, with the Duo passcode, that comes from our Vault instance includes the IP address of the host system so that it can a) be blocked if it is not in the whitelisted range and b) be recorded in the Duo audit logs. Currently there is no workaround for this.

We have this PR. There is also a similar PR from Duo: [here]("Networks for API access" feature for Auth API)

tl;dr: We want Vault to include it's source IP address when sending any Duo request

@finnstech
Copy link
Contributor

@gw-uber thanks for posting this request and also the related PR. I've added them to Jira so we can evaluate them as part of the MFA improvements for upcoming releases. We will keep you posted about this issue.

@heatherezell
Copy link
Contributor

This has been addressed in: #18811
Thanks for bringing it up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants