Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ETF Gateway leaking Guilds.Models.RecurrenceRule #6580

Closed
Th3-M4jor opened this issue Dec 18, 2023 · 6 comments
Closed

ETF Gateway leaking Guilds.Models.RecurrenceRule #6580

Th3-M4jor opened this issue Dec 18, 2023 · 6 comments
Assignees
Labels
bug synced Synced to internal tracker

Comments

@Th3-M4jor
Copy link
Contributor

Description

Similar to #4196 and #6320, another :__struct__ field is being leaked by the gateway, Guilds.Models.RecurrenceRule this time.

Steps to Reproduce

Currently unclear, I'm guessing this field is part of a GUILD_UPDATE event. It was reported for the Nostrum Library here.

Expected Behavior

No :__struct__ fields are sent as part of gateway events

Current Behavior

An event may include __struct__: Guilds.Models.RecurrenceRule in its body

Screenshots/Videos

No response

Client and System Information

Nostrum library on its latest commit.

@Th3-M4jor Th3-M4jor added the bug label Dec 18, 2023
@mbialecka mbialecka added the synced Synced to internal tracker label Dec 19, 2023
@Leastrio
Copy link

Ive recently been getting Guilds.Models.ScheduledEventException leaked.

@jb3
Copy link
Contributor

jb3 commented May 7, 2024

Heads up we still have Nostrum users receiving these structs.

It's not critical for us anymore since we added protections to remove struct keys received from upstream, but does remain a bug.

@ihumanable
Copy link

@jb3

Heads up we still have Nostrum users receiving these structs.

It's not critical for us anymore since we added protections to remove struct keys received from upstream, but does remain a bug.

Do you know which struct is being sent over the gateway? Is it still the Guilds.Models.ScheduledEventException or some other struct key?

@jb3
Copy link
Contributor

jb3 commented May 7, 2024

Do you know which struct is being sent over the gateway? Is it still the Guilds.Models.ScheduledEventException or some other struct key?

The one reported today is the ScheduledEventException, testing now it looks like recurrence rule seems to be okay (at least on scheduled event create, not sure where else that struct might be serialized over gateway).

1 similar comment
@jb3
Copy link
Contributor

jb3 commented May 7, 2024

Do you know which struct is being sent over the gateway? Is it still the Guilds.Models.ScheduledEventException or some other struct key?

The one reported today is the ScheduledEventException, testing now it looks like recurrence rule seems to be okay (at least on scheduled event create, not sure where else that struct might be serialized over gateway).

@ollien
Copy link

ollien commented Jun 19, 2024

We have deployed a fix to prevent the __struct__ key from being leaked for Guilds.Models.ScheduledEventException.

Thanks so much for bringing this to our attention!

@ollien ollien closed this as completed Jun 19, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug synced Synced to internal tracker
Projects
None yet
Development

No branches or pull requests

6 participants