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.

Sign up for GitHub

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

Jump to bottom

chore: lower dhi to limit amplification factor #850

Merged
merged 2 commits into from
Oct 31, 2023
Merged

chore: lower dhi to limit amplification factor #850

merged 2 commits into from
Oct 31, 2023

Conversation

richard-ramos
Copy link
Member

Description

Matches waku-org/nwaku#2168

@richard-ramos richard-ramos changed the title chore: lower dhigh to limit amplification factor chore: lower dhi to limit amplification factor Oct 30, 2023
@status-im-auto
Copy link

status-im-auto commented Oct 30, 2023 •
edited
Loading

Jenkins Builds

❔ Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ d693e49 #1 2023-10-30 18:24:33 ~2 min nix-flake 📄log
✔️ d693e49 #1 2023-10-30 18:24:53 ~2 min linux 📦deb
✔️ d693e49 #1 2023-10-30 18:25:39 ~3 min tests 📄log
✔️ d693e49 #1 2023-10-30 18:26:00 ~3 min tests 📄log
✔️ d693e49 #1 2023-10-30 18:26:32 ~4 min android 📦tgz
✔️ d693e49 #1 2023-10-30 18:39:39 ~17 min ios 📦tgz
✖️ f25031f #2 2023-10-31 10:52:03 ~1 min tests 📄log
✔️ f25031f #2 2023-10-31 10:52:42 ~1 min nix-flake 📄log
✔️ f25031f #2 2023-10-31 10:53:14 ~2 min linux 📦deb
✔️ f25031f #2 2023-10-31 10:54:23 ~3 min tests 📄log
✔️ f25031f #2 2023-10-31 10:54:33 ~3 min android 📦tgz
✔️ f25031f #3 2023-10-31 10:56:25 ~2 min tests 📄log
✔️ f25031f #2 2023-10-31 11:03:11 ~12 min ios 📦tgz

@richard-ramos richard-ramos added the E:1.1 Network requirements and task breakdown See https://github.com/waku-org/pm/issues/62 for details label Oct 30, 2023
chaitanyaprem
chaitanyaprem approved these changes Oct 30, 2023
View reviewed changes
Copy link
Collaborator

@chaitanyaprem chaitanyaprem left a comment

Choose a reason for hiding this comment

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

LGTM

I am wondering if we should start having these as part of config profiles.

@richard-ramos richard-ramos merged commit 5dfbd98 into master Oct 31, 2023
11 checks passed
@richard-ramos richard-ramos added E:3.2: Basic DoS protection in production See https://github.com/waku-org/pm/issues/70 for details and removed E:1.1 Network requirements and task breakdown See https://github.com/waku-org/pm/issues/62 for details labels Nov 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chaitanyaprem chaitanyaprem chaitanyaprem approved these changes

@harsh-98 harsh-98 Awaiting requested review from harsh-98

Assignees

@richard-ramos richard-ramos

Labels
E:3.2: Basic DoS protection in production See https://github.com/waku-org/pm/issues/70 for details
Projects
Waku
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@richard-ramos @status-im-auto @chaitanyaprem