Skip to content

Randomly pad OHTTP requests#715

Merged
spacebear21 merged 1 commit intopayjoin:masterfrom
nothingmuch:random-padding
May 22, 2025
Merged

Randomly pad OHTTP requests#715
spacebear21 merged 1 commit intopayjoin:masterfrom
nothingmuch:random-padding

Conversation

@nothingmuch
Copy link
Collaborator

Using random padding instead of 0 padding allows for multi-hop OHTTP to use pre-computed filler strings similarly to Sphinx indistinguishably from single hop requests which just use random padding.

Making this change now means that if we ever implement multi-hop requests, their use would not be observable by the directory.

Using random padding instead of 0 padding allows for multi-hop OHTTP
to use pre-computed filler strings similarly to Sphinx indistinguishably
from single hop requests which just use random padding.

Making this change now means that if we ever implement multi-hop
requests, their use would not be observable by the directory.
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 15189764470

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.003%) to 83.528%

Totals Coverage Status
Change from base Build 15169328807: 0.003%
Covered Lines: 5948
Relevant Lines: 7121

💛 - Coveralls

Copy link
Collaborator

@spacebear21 spacebear21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

Copy link
Collaborator

@arminsabouri arminsabouri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Concept ACK

@spacebear21 spacebear21 merged commit a5dba58 into payjoin:master May 22, 2025
7 checks 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.

4 participants