Browse Source

fix: strip trailing slash from ratelimit bucket IDs

This resolves #1125.

fixes a bug where some ratelimit buckets would include a trailing slash,
while others wouldn't; this would cause them to be treated as separate
ratelimits, even though they are the same

ideally this fix should change the ratelimit generator, but that code is
pretty complicated and this was an easier fix that seems less likely to
break things in the future.

tested against normal bot function, all routes are assigned the proper
buckets from my testing, so this should be good to go.
pull/1163/head
Christopher F 7 years ago
parent
commit
f8d1cd6c85
1 changed files with 3 additions and 0 deletions
  1. +3
    -0
      src/Discord.Net.Rest/DiscordRestApiClient.cs

+ 3
- 0
src/Discord.Net.Rest/DiscordRestApiClient.cs View File

@@ -1415,6 +1415,9 @@ namespace Discord.API
}

format = builder.ToString();
if (format.LastIndexOf('/') == format.Length-1)
format = format.Substring(0, format.Length - 1); // perf: change the code above so this isn't necessary

return x => string.Format(format, x.ToArray());
}
catch (Exception ex)


Loading…
Cancel
Save