Follow-up from "Workhorse: HTTP endpoint for bidirectional SSH git-upload-pack data"
<!--IssueSummary start--> <details> <summary> Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards. </summary> - [Close this issue](https://contributors.gitlab.com/manage-issue?action=close&projectId=278964&issueIid=467288) </details> <!--IssueSummary end--> The following discussion from !152950 should be addressed: - [ ] @ashmckenzie started a [discussion](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/152950#note_1947956653): > suggestion (nitpick): We could early return and save some indentation etc: > > ```suggestion:-6+0 > return super unless geo? > > set_workhorse_internal_api_content_type > > render json: ::Gitlab::Workhorse.git_http_ok(repository, repo_type, user, :git_upload_pack, show_all_refs: geo_request?, need_audit: need_git_audit_event?) > ``` > > This could be a follow-up as it's very nitty 🙂
issue