server: Add recursion limit to resolver #261

Merged

norm merged 3 commits from recursion-limit into main 2022-12-01 20:11:41 +00:00

Conversation 0 Commits 3 Files changed 4 +13 -9

norm commented 2022-12-01 09:39:52 +00:00

Owner

Copy link

Patches pulled from https://code.vtopia.live/Vtopia/MissV/pulls/8

This mitigates an attack where an actor object can recursively reference randomly generated actors, which can cause the instance to slow to a crawl.

Patches pulled from https://code.vtopia.live/Vtopia/MissV/pulls/8 This mitigates an attack where an actor object can recursively reference randomly generated actors, which can cause the instance to slow to a crawl.

norm force-pushed recursion-limit from dab5efbc02 to 11a6e706f4 2022-12-01 09:40:20 +00:00 Compare

Johann150 added 1 commit 2022-12-01 19:34:09 +00:00

use default argument value ... 936cbf900b

This unifies the style with the other function in that file and fixes
the lint "no-param-reassign".

Johann150 approved these changes 2022-12-01 19:35:59 +00:00

Johann150 left a comment

Owner

Copy link

Looks good to me now.

Looks good to me now.

Johann150 referenced this pull request 2022-12-01 19:42:02 +00:00

server: check that recursion limit is used everywhere #264

norm merged commit dc7533baa4 into main 2022-12-01 20:11:41 +00:00

norm deleted branch recursion-limit 2022-12-01 20:11:41 +00:00

norm referenced this pull request from a commit 2022-12-01 20:11:41 +00:00

Merge pull request 'server: Add recursion limit to resolver' (#261) from recursion-limit into main

Sign in to join this conversation.

Reviewers

No reviewers

Johann150

Labels

Clear labels   

feature

task that proposes new feature(s)

  

fix

task to fix behavioral problems in the FoundKey codebase

  

upkeep

task that should make FoundKey smaller and more maintainable

No labels

feature

fix

upkeep

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: FoundKeyGang/FoundKey#261

No description provided.