Closes #37436
Closes gitaly#998

This reverts commit a58f8c32, reversing
changes made to cd5d75c3.

Fix using wildcards in protected tags to expose protected variables

Merge branch 'security-10-4-25223-snippets-finder-doesnt-obey-feature-visibility' into 'security-10-4'

[Port for security-10-4]: Makes SnippetFinder ensure feature visibility

MergeRequest#rebase_in_progress? and MergeRequest#rebase_path were called
twice each time per request. This memoization helps reduce the overall number
of queries.

See https://gitlab.com/gitlab-org/gitlab-ee/-/jobs/51616319 as a failing
job.

Closes gitlab-org/gitlab-ee#4857

This reduces the number of RefExists RPC call.

+ address small nitpicks

- Move the exception handling as close to the source as possible to
avoid leaking Psych ahstraction
- Also remove unnecessary rescue all statement from LintsController.
This should not be necessary anymore since any YAML errors should all be
caught by the #validation_message method.

Signed-off-by: Bastian Blank <waldi@debian.org>

it will also automatically prune empty directories for
`FileUploader`-based uploaders.

Removing last_sign_in_at in specs

Fixes #32282.