-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
TT-1740 Adding fallback scenario for ctx Get Apidef and Session #3875
Conversation
API tests result: success ✅ |
API tests result: success ✅ |
@tbuchaillot if this doesn't panic how will the user know that its a compiler version issue?does this display any log? |
@sredxny I added a log but it shouldn't happen. It's just a fallback scenario based on #3612 (comment) workaround for when this error was present. |
API tests result: success ✅ |
API tests result: success ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
API tests result: success ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
API tests result: skipped 🚫 |
2 similar comments
API tests result: skipped 🚫 |
API tests result: skipped 🚫 |
SonarCloud Quality Gate failed. |
API tests result: skipped 🚫 |
/release to release-4 |
API tests result: skipped 🚫
|
/release to release-4 |
Working on it! Note that it can take a few minutes. |
@tbuchaillot Succesfully merged |
* adding fallback scenario for missversions in ctx Apidef and Session * Adding test to check go plugin API ctx * adding log in unmarshalling (cherry picked from commit 44d50eb)
/release to release-4-lts |
Working on it! Note that it can take a few minutes. |
* adding fallback scenario for missversions in ctx Apidef and Session * Adding test to check go plugin API ctx * adding log in unmarshalling (cherry picked from commit 44d50eb)
@tbuchaillot Succesfully merged |
Description
Adding a fallback scenario to avoid panics in plugin developments where the plugin compiler has missversion.
This
Related Issue
https://tyktech.atlassian.net/browse/TT-1740
Motivation and Context
How This Has Been Tested
Screenshots (if appropriate)
Types of changes
Checklist
fork, don't request your
master
!master
branch (left side). Also, you should startyour branch off our latest
master
.go mod tidy && go mod vendor
go fmt -s
go vet