Skip to content

Consolidate Puma low-level handler code

Stan Hu requested to merge sh-consolidate-puma-lowlevel-handler into main

What does this merge request do and why?

Previously the Puma low-level error handler was specified in three different places (Cloud Native GitLab, Omnibus GitLab, GitLab Development Kit). Any time a change was needed we would have to update three different files. We can simplify this by putting in a central class that each of the configuration files can use.

This change requires gitlab!132581 (merged).

Relates to gitlab#426135 (closed)

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

Impacted categories

The following categories relate to this merge request:

Merge request checklist

  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Tests added for new functionality. If not, please raise an issue to follow-up.
  • Documentation added/updated, if needed.
  • Announcement added, if change is notable.
  • gdk doctor test added, if needed.
  • Add the ~highlight label if this MR should be included in the CHANGELOG.md.
Edited by Stan Hu

Merge request reports