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.

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

GitHub Comment Length Limit #919

Open
fbartho opened this issue Sep 19, 2019 · 6 comments
Open

GitHub Comment Length Limit #919

fbartho opened this issue Sep 19, 2019 · 6 comments
Assignees

Comments

@fbartho
Copy link
Member

fbartho commented Sep 19, 2019

The GitHub Integration just fails with a GitHub Error Response if your Issue Comment gets too long.

It’d be great if we truncated the message, & added a [truncated] 📝?

Thoughts @orta ? I’m happy to implement this if you have advice!

@fbartho fbartho self-assigned this Sep 19, 2019
@orta
Copy link
Member

orta commented Sep 19, 2019

I think it should already do that, maybe there's some edge cases where it doesn't?

@fbartho
Copy link
Member Author

fbartho commented Sep 19, 2019

Is it possible that github’s limit changed?

@orta
Copy link
Member

orta commented Sep 19, 2019

That seems unlikely to me, given how much stuff could break - check the source and your repro, to see if you can think of a way?

@fbartho
Copy link
Member Author

fbartho commented Sep 19, 2019

I looked through each of the files under the Github Platform, and I couldn't find any that did truncation? Do you have an idea where the implementation of that truncation might be?

I mostly looked here: https://github.com/danger/danger-js/blob/master/source/platforms/github

I also searched our code for 65536 and I couldn't find that constant anywhere except as a spurious power-of-two constant in an old Issue (unrelated).

@fbartho
Copy link
Member Author

fbartho commented Sep 19, 2019

{ fails: { numEntries: 3, sumNumChars: 272745 },
  warnings: { numEntries: 0, sumNumChars: 0 },
  messages: { numEntries: 1, sumNumChars: 123 },
  markdowns: { numEntries: 0, sumNumChars: 0 } }

Looking at this debug log I stuck in, it looks like there were a ton of text in one of my fail messages. (I think it's a lot of snapshot failures)

@orta
Copy link
Member

orta commented Sep 25, 2019

Ah, maybe it was in danger ruby we built infra to check the length. Hard to keep on top of all the implementations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants