ChannelManager.add calling on a deleted channel creating error #5635

Evercreeper · 2021-05-15T05:03:29Z

Please describe the problem you are having in as much detail as possible:
Hey. I host a bot with channel-creation capabilities for a discord server. It is hosted off of a Replit IDE

Recently, our bot has been getting Discord.js internal errors that we cannot catch regarding our channel creation system. (this specific error message from our test build)

/home/runner/RPM-TEST-BUILD/node_modules/discord.js/src/managers/ChannelManager.js:23
    const existing = this.cache.get(data.id);
                                         ^

TypeError: Cannot read property 'id' of undefined
    at ChannelManager.add (/home/runner/RPM-TEST-BUILD/node_modules/discord.js/src/managers/ChannelManager.js:23:42)
    at Invite._patch (/home/runner/RPM-TEST-BUILD/node_modules/discord.js/src/structures/Invite.js:95:41)
    at new Invite (/home/runner/RPM-TEST-BUILD/node_modules/discord.js/src/structures/Invite.js:15:10)
    at InviteCreateAction.handle (/home/runner/RPM-TEST-BUILD/node_modules/discord.js/src/client/actions/InviteCreate.js:15:20)
    at Object.module.exports [as INVITE_CREATE] (/home/runner/RPM-TEST-BUILD/node_modules/discord.js/src/client/websocket/handlers/INVITE_CREATE.js:4:31)
    at WebSocketManager.handlePacket (/home/runner/RPM-TEST-BUILD/node_modules/discord.js/src/client/websocket/WebSocketManager.js:386:31)

Now at first, I thought it could be OUR code. The latest error happened when someone did our creation command with a bunch of emojis "-uca text 60 😀😀😀😀😀😀😀".
I figured there was a problem with channel creation. Then after figuring out it was not on my end, I actually looked at the error better.

My UCA content is a category which is at the TOP of our channel list. This means any defaulted invites that go above "uca-info" may be sent to a deleted channel!

Image: https://prnt.sc/12y6tie

Now lets go look at the code featured in the ChannelManager:

add(data, guild, cache = true) {
    const existing = this.cache.get(data.id); // As you can see, a channel should always exist when this is called. However, when Discord defaulting picks the invite, it can be to a channel that no longer exists, therefore no .id
    if (existing) {
      if (existing._patch && cache) existing._patch(data);
      if (guild) guild.channels.add(existing);
      return existing;
    }

    const channel = Channel.create(this.client, data, guild);

    if (!channel) {
      this.client.emit(Events.DEBUG, `Failed to find guild, or unknown type for channel ${data.id} ${data.type}`);
      return null;
    }

    if (cache) this.cache.set(channel.id, channel);

    return channel;
  }

Now I am no expert, this being my first ever github issue I have created, how ever I do not know if this is a Discord API issue or a Discord.js code issue. As I put in the comment, it seems that this is being called with no data and therefore the id property is absent.

I do not really know anything past what I speculate here. It seems to be an invite issue, but how could a default invite be invoked when I don't call on Discord.js to handle an invite? Does discord.js handle invites?

Anyhow I appreciate you taking your time to deal with this issue. Again, I am sorry for not being super pro who can automatically find what part is wrong, but hopefully the information I provided is beneficial.

Further details:

discord.js version: 12.5.3
Node.js version: 16.1.0
Operating system: No idea, it is ran on Replit so I assume some sort of VM
Priority this issue should have : Medium priority - Messing with channels at the top of the guild isn't done a lot, but this seemingly random error causes a lot of issues when you try to have a 24/7 bot.

Evercreeper · 2021-05-15T05:05:53Z

Sorry if I broke any formats. Currently 1:00 AM for me and I am just trying to report this issue as best as I can. If you do want my specific code to verify it is not on my end, please tell me and I will provide right away.

Evercreeper · 2021-05-17T04:15:53Z

After further investigation, I am 99% sure this is an internal discord.js error and should have a higher priority.

I am currently running uptime tests and I am going back checking the last bot activity when it crashes from this error. This is the only error currently crashing my bot. After some tests, I found out that my channel creator WAS NOT at fault here.

My bot would crash to this error after a few things such as:

Welcoming members
Posting messages in support tickets
Creating channels

Now that we know it is not on my end, what could be causing this issue? As I stated in my original issue, I haven't done my fair share of investigating internally and I don't know what each part of the error is.

monbrey · 2021-05-17T04:30:10Z

As far as I can tell looking at this

You're getting an INVITE_CREATE event via the websocket
This calls the Action handler
This creates a new Invite
This calls ChannelManager#add for invite.channel
Somehow, invite.channel was null and it errors

This should be impossible according to the API docs for an Invite - channel is not optional
https://discord.com/developers/docs/resources/invite#invite-object-invite-structure

Evercreeper · 2021-05-20T04:02:36Z

Yeah, this seems to be really strange. Thank you for the info. Is there a way I can catch the discord.js error to at least maintain the uptime of the bot while I figure out the root cause?

I looked at the errors guide and the closest thing I set up was a shardError and it does not catch this, although the error is really in the module and not in the websocket itself so that is expected.

Sorry again if I am missing any formatting or I am just being generally unhelpful here, first time making an issue and first time dealing with an impossible bug that I can't find out what is wrong.

TranquillyUnpleasant · 2021-05-25T08:54:50Z

Ive had the same error happening. Since the bot doesn't do anything with invites im fairly certain the error is within the library itself. The stack trace also points to such. Here is the trace:

/home/io/discordbot/ModBotv2/node_modules/discord.js/src/managers/ChannelManager.js:23
    const existing = this.cache.get(data.id);
                                         ^

TypeError: Cannot read property 'id' of undefined
    at ChannelManager.add (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/managers/ChannelManager.js:23:42)
    at Invite._patch (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/structures/Invite.js:95:41)
    at new Invite (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/structures/Invite.js:15:10)
    at InviteDeleteAction.handle (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/actions/InviteDelete.js:15:20)
    at Object.module.exports [as INVITE_DELETE] (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/websocket/handlers/INVITE_DELETE.js:4
:31)
    at WebSocketManager.handlePacket (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/websocket/WebSocketManager.js:384:31)
    at WebSocketShard.onPacket (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/websocket/WebSocketShard.js:444:22)
    at WebSocketShard.onMessage (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/websocket/WebSocketShard.js:301:10)
    at WebSocket.onMessage (/home/io/discordbot/ModBotv2/node_modules/ws/lib/event-target.js:132:16)
    at WebSocket.emit (events.js:314:20)
Thrown at:
    at add (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/managers/ChannelManager.js:23:42)
    at _patch (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/structures/Invite.js:95:41)
    at Invite (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/structures/Invite.js:15:10)
    at handle (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/actions/InviteDelete.js:15:20)
    at module.exports (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/websocket/handlers/INVITE_DELETE.js:4:31)
    at handlePacket (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/websocket/WebSocketManager.js:384:31)
    at onPacket (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/websocket/WebSocketShard.js:444:22)
    at onMessage (/home/io/discordbot/ModBotv2/node_modules/discord.js/src/client/websocket/WebSocketShard.js:301:10)
    at onMessage (/home/io/discordbot/ModBotv2/node_modules/ws/lib/event-target.js:132:16)
    at emit (events.js:314:20)

Notably, its on the invite delete event in this case, which only happened within this guild due to a timeout, no invites have been manually deleted, and the bot is only in one guild.

charlieTheBotDev · 2021-05-26T09:40:20Z

Also seeing this issue today repeatedly and it's crashing my bot with seemingly no way to handle it.

The crash seems to be more like an infinite loop / unresponsive error rather than a standard process crash (which I handle and auto-restart) so the bot just hangs indefinitely.

[RGB] (2021-05-26T09:12:57.968Z) Process: Uncaught Exception TypeError: Cannot read property 'id' of undefined
    at ChannelManager.add (/root/git/rgb/bot/node_modules/discord.js/src/managers/ChannelManager.js:23:42)
    at Invite._patch (/root/git/rgb/bot/node_modules/discord.js/src/structures/Invite.js:95:41)
    at new Invite (/root/git/rgb/bot/node_modules/discord.js/src/structures/Invite.js:15:10)
    at InviteDeleteAction.handle (/root/git/rgb/bot/node_modules/discord.js/src/client/actions/InviteDelete.js:15:20)
    at Object.module.exports [as INVITE_DELETE] (/root/git/rgb/bot/node_modules/discord.js/src/client/websocket/handlers/INVITE_DELETE.js:4:31)
    at WebSocketManager.handlePacket (/root/git/rgb/bot/node_modules/discord.js/src/client/websocket/WebSocketManager.js:384:31)
    at WebSocketShard.onPacket (/root/git/rgb/bot/node_modules/discord.js/src/client/websocket/WebSocketShard.js:444:22)
    at WebSocketShard.onMessage (/root/git/rgb/bot/node_modules/discord.js/src/client/websocket/WebSocketShard.js:301:10)
    at WebSocket.onMessage (/root/git/rgb/bot/node_modules/ws/lib/event-target.js:132:16)
    at WebSocket.emit (events.js:315:20)

I'm getting this from process.on('uncaughtException', (error) => console.log(...)

I'm listening to the following events from client:

guildDelete
guildCreate
warn
error
rateLimit
error
shardError
message

None of which seem to be triggering the issue.

I was on 12.5.1 but have updated to 12.5.3 to see if the issue goes away (although as the OP was using 12.5.3 I doubt it)

I suspect it's to do with inviting a bot to a stage channel but I can't seem to do that on my server so 🤷

Edit:
I've tried to catch it with every event handler to no avail. Seems that the issue may be related to a change in the Discord API which as they have been touching Stage channels recently would make sense

I've manually patched my node_modules with a small if statement inside the top of add for node_modules/discord.js/src/managers/ChannelManager.js

  add(data, guild, cache = true) {
    if (!data) {
      this.client.emit(Events.DEBUG, `No data provided`);
      return null;
    }

(Basically a copy of the !channel block so hopefully won't cause any problems)

For reference, the reason it makes your bot 'hang' is because the webhook handler never completes (i.e. it doesn't return anything) so simply adding the return null; is sufficient, I added the debug log so I know when it happens :)

Evercreeper · 2021-05-30T20:59:31Z

As you say @c-frater, it really is an easy fix like that.
However, this still doesn't stop this rogue invite-create call and I fear until someone finds out what is exactly going wrong, this wont be publicly patched.

I think for now I will have to do the same as you and change our node_modules. Thanks for the fix 👍

charlieTheBotDev · 2021-05-30T21:49:35Z

Yeah, just for complete explicitness this fixes the symptom of whatever is calling that rouge invite-create and most definitely needs to be fixed at the source in D.js rather than applying my change blindly and hiding the real issue away.

Shared purely as a workaround until they work out the root cause 👍

ExpErgio · 2021-06-09T09:05:18Z

I need help. I did all @c-frater said in node_modules/discord.js/src/managers/ChannelManager.js

'use strict';

const BaseManager = require('./BaseManager');
const Channel = require('../structures/Channel');
const { Events } = require('../util/Constants');

/**
 * A manager of channels belonging to a client
 * @extends {BaseManager}
 */
class ChannelManager extends BaseManager {
  constructor(client, iterable) {
    super(client, iterable, Channel);
  }

  /**
   * The cache of Channels
   * @type {Collection<Snowflake, Channel>}
   * @name ChannelManager#cache
   */

  add(data, guild, cache = true) {
    if(!data) { // ---------------------------------------------   HERE
      this.client.emit(Events.DEBUG, `No data provided`);
      return null;
    } // --------------------------------------------------------  HERE
    const existing = this.cache.get(data.id);
    if (existing) {
      if (existing._patch && cache) existing._patch(data);
      if (guild) guild.channels.add(existing);
      return existing;
    }

but it doesnt work. When I place my bot in a Stage Channel, it crash. In my console, I have this error:

C:\Users\Usuario\Desktop\BOT_Alejandreta 2.0\node_modules\discord.js\src\client\voice\networking\VoiceWebSocket.js:123
        server_id: this.connection.channel.guild.id,
                                           ^

TypeError: Cannot read property 'guild' of undefined
    at VoiceWebSocket.onOpen (C:\Users\Usuario\Desktop\BOT_Alejandreta 2.0\node_modules\discord.js\src\client\voice\networking\VoiceWebSocket.js:123:44)
    at WebSocket.onOpen (C:\Users\Usuario\Desktop\BOT_Alejandreta 2.0\node_modules\ws\lib\event-target.js:144:16)
    at WebSocket.emit (events.js:315:20)
    at WebSocket.setSocket (C:\Users\Usuario\Desktop\BOT_Alejandreta 2.0\node_modules\ws\lib\websocket.js:177:10)
    at ClientRequest.<anonymous> (C:\Users\Usuario\Desktop\BOT_Alejandreta 2.0\node_modules\ws\lib\websocket.js:671:15)
    at ClientRequest.emit (events.js:315:20)
    at TLSSocket.socketOnData (_http_client.js:547:11)
    at TLSSocket.emit (events.js:315:20)
    at addChunk (internal/streams/readable.js:309:12)
    at readableAddChunk (internal/streams/readable.js:284:9)
    at TLSSocket.Readable.push (internal/streams/readable.js:223:10)
    at TLSWrap.onStreamRead (internal/stream_base_commons.js:188:23)

SpaceEEC · 2021-06-10T12:24:14Z

This should have been fixed by #4882, can somebody provide some steps to reproduce this error?

charlieTheBotDev · 2021-06-10T13:51:01Z

@ExpErgio the issue you're having is unrelated to ChannelManager.add - It may be a new/already reported issue elsewhere

Evercreeper · 2021-06-13T23:49:00Z

This should have been fixed by #4882, can somebody provide some steps to reproduce this error?

This is exactly my problem. There IS no way to reproduce this error and seems to happen at random. I can provide some more information about the server itself, but other than every since I forked and did c-frater's solution I have completed stopped the error.

The bot is in only one server.
This server is a public community server that has ~20k members.
Any user can create invite links.

I wish there were more to say but unfortunately I do not have a definite answer. I must say however, this error did not start occurring until about April.

SpaceEEC · 2021-06-14T19:15:25Z

I'm really puzzled about how that could be happening, since we explicitly check for the channel's truthiness in both of the action handlers. 😅

Can you log the data along it's way to the error? Maybe that could give us any clues about why this is happening.

Relevant places (although those are already visible in the stacktrace):

discord.js/src/client/actions/InviteCreate.js

Line 15 in 51551f5

const invite = new Invite(client, inviteData);

discord.js/src/client/actions/InviteDelete.js

Line 15 in 51551f5

const invite = new Invite(client, inviteData);

discord.js/src/structures/Invite.js

Line 15 in 51551f5

this._patch(data);

discord.js/src/structures/Invite.js

Line 95 in 51551f5

this.channel = this.client.channels.add(data.channel, this.guild, false);

discord.js/src/managers/ChannelManager.js

Line 23 in 51551f5

const existing = this.cache.get(data.id);

iCrawl · 2021-07-24T00:14:11Z

Closing because stale, open a new issue if this is still a problem.

Evercreeper added s: unverified labels May 15, 2021

iCrawl closed this as completed Jul 24, 2021

github-actions bot locked as resolved and limited conversation to collaborators Mar 30, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ChannelManager.add calling on a deleted channel creating error #5635

ChannelManager.add calling on a deleted channel creating error #5635

Evercreeper commented May 15, 2021

Evercreeper commented May 15, 2021

Evercreeper commented May 17, 2021

monbrey commented May 17, 2021

Evercreeper commented May 20, 2021

TranquillyUnpleasant commented May 25, 2021 •

edited

Loading

charlieTheBotDev commented May 26, 2021 •

edited

Loading

Evercreeper commented May 30, 2021

charlieTheBotDev commented May 30, 2021

ExpErgio commented Jun 9, 2021

SpaceEEC commented Jun 10, 2021

charlieTheBotDev commented Jun 10, 2021

Evercreeper commented Jun 13, 2021 •

edited

Loading

SpaceEEC commented Jun 14, 2021

iCrawl commented Jul 24, 2021

ChannelManager.add calling on a deleted channel creating error #5635

ChannelManager.add calling on a deleted channel creating error #5635

Comments

Evercreeper commented May 15, 2021

Evercreeper commented May 15, 2021

Evercreeper commented May 17, 2021

monbrey commented May 17, 2021

Evercreeper commented May 20, 2021

TranquillyUnpleasant commented May 25, 2021 • edited Loading

charlieTheBotDev commented May 26, 2021 • edited Loading

Evercreeper commented May 30, 2021

charlieTheBotDev commented May 30, 2021

ExpErgio commented Jun 9, 2021

SpaceEEC commented Jun 10, 2021

charlieTheBotDev commented Jun 10, 2021

Evercreeper commented Jun 13, 2021 • edited Loading

SpaceEEC commented Jun 14, 2021

iCrawl commented Jul 24, 2021

TranquillyUnpleasant commented May 25, 2021 •

edited

Loading

charlieTheBotDev commented May 26, 2021 •

edited

Loading

Evercreeper commented Jun 13, 2021 •

edited

Loading