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

Fix: ruby imports #271

Merged
merged 3 commits into from
Jul 26, 2024
Merged

Fix: ruby imports #271

merged 3 commits into from
Jul 26, 2024

Conversation

janz93
Copy link
Contributor

@janz93 janz93 commented Jul 26, 2024

What changes are introduced?

  1. Fix broken ruby imports
  2. Add HTTP status 500 to swagger

Why are these changes introduced?

  1. Due to the broken imports it was not possible to create new credit_transfers and direct_debits as of version 2.0
  2. Any HTTP 500 result broke the swagger client since that was an unknown response

How are these changes made?

  1. Make explicit imports to fix the import hierarchy
  2. Add 500 as valid response to swagger

How was it tested? (optional)

remove this section, when you don't add further information

Some code, especially infrastructure code (say HELM or Kubernetes yaml files) are harder to test. So it’s important to let the reviewer know how you tested them in case you can’t check in tests. Alternatively, you can explain to the reviewer how to test it locally if necessary. Showing the results of tests you’ve run in this section if none are visible in the diff is also very helpful.

  • Specs
  • Locally
  • Staging

Copy link
Member

@bonflintstone bonflintstone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks quite good

In swagger we did not define 500 as a valid response and they current
response was not returning `json` in the body.
This broke the swagger UI and could break other clients as well.

Note: To prevent any exception exploiting we only revels the error to
the configured error tracking system and the server logs.
With ruby 3 the import logic was adjusted which broke the current import
hierarchy.
@janz93 janz93 enabled auto-merge (rebase) July 26, 2024 13:41
@janz93 janz93 merged commit 6ca2cec into main Jul 26, 2024
3 checks passed
@janz93 janz93 deleted the fix/ruby-imports branch July 26, 2024 13:41
@janz93 janz93 changed the title Fix/ruby imports Fix: ruby imports Jul 26, 2024
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