Skip to content
This repository has been archived by the owner on Aug 28, 2023. It is now read-only.

origin can be used to trigger a call from the client gateway to anywhere #275

Closed
rmeissner opened this issue Feb 17, 2021 · 2 comments · Fixed by #297
Closed

origin can be used to trigger a call from the client gateway to anywhere #275

rmeissner opened this issue Feb 17, 2021 · 2 comments · Fixed by #297
Assignees
Labels
bug Something isn't working

Comments

@rmeissner
Copy link
Member

rmeissner commented Feb 17, 2021

  • Check if this is a security issue
    • Could you create a loop where the origin points to the safe transaction itself and therefore results in the interface calling itself?
  • In some cases this caused a blocking call (as we did not have a connection timeout) which at some point blocked the whole service.
    • Switch to non blocking version of reqwest

Solutions:

  • Disallow localhost as source
  • Disallow static IPs as source (aka require a dns entry)
  • Only allow http protcols
@rmeissner rmeissner added the bug Something isn't working label Feb 17, 2021
@jpalvarezl
Copy link
Contributor

For non blocking reqwest migration, it could be worth checking out this: rwf2/Rocket#1065

@rmeissner
Copy link
Member Author

We do not need to wait for this, as according to the rocket docs they already support async io (see https://rocket.rs/master/guide/overview/#futures-and-async). This still would be a major change that would affect the whole app.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants