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

change labels id to long in github #399

Merged
merged 2 commits into from
Aug 26, 2020
Merged

Conversation

ofersk
Copy link
Contributor

@ofersk ofersk commented Aug 25, 2020

Description

In preparing for a POC, we encountered an issue with Github repository enabled with Dependabot security (Github SCA solution); this is used by the prospect for SCA

During the parsing of existing Github issues, error was flagged while extracting the response - JsonMappingException: Numeric value (2175583853) out of range of int (-2147483648 - 2147483647)

References

Cases 00056883

Testing

N/A

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable). If documentaiton is a Wiki Update, please indicate desired changes within PR MD Comment
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@ofersk ofersk requested a review from AvivCx August 25, 2020 18:25
@sonarcloud
Copy link

sonarcloud bot commented Aug 25, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

warning The version of Java (1.8.0_265) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@ofersk ofersk merged commit 908bfd4 into develop Aug 26, 2020
@ofersk ofersk deleted the pr-ofer@bug_github_issue_label_id branch August 26, 2020 06:14
james-bostock-cx pushed a commit to james-bostock-cx/cx-flow that referenced this pull request Mar 18, 2021
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

Successfully merging this pull request may close these issues.

2 participants