Some API requests are handled by Grape directly without ever going
through `before` blocks. We can still log `remote_ip` (and hence
`client_id`) for those requests, as that doesn't depend on anything
application-specific.

We handle this by putting it in the logger itself as well as the
`before` block. This is because `client_id` is computed based on an
ApplicationContext instance, but this is backed by a Labkit::Context. A
Labkit::Context isn't aware of our models, and so `user` is only
inherited as a string, not an object - which means that `client_id`
could end up incorrectly using the IP rather than the user ID.

Doc Consistency: user/analytics - release

See merge request gitlab-org/gitlab!72147

Doc Consistency: user/analytics - optimize

See merge request gitlab-org/gitlab!72148

Remove feature flag for CTA in empty state

See merge request gitlab-org/gitlab!71952

Correct metadata of external auth doc

See merge request gitlab-org/gitlab!72077

Rename profile password fields so password managers understand

See merge request gitlab-org/gitlab!71237

Typo fixed (needless repetition)

See merge request gitlab-org/gitlab!72154

Updated nav steps

See merge request gitlab-org/gitlab!72047

Add phone to user_details and create verification_codes

See merge request gitlab-org/gitlab!71139

[ci skip]

Docs: correct link to Windows CI image software

See merge request gitlab-org/gitlab!72155

Fix Cross-DB transaction on Deployment Status Sync

See merge request gitlab-org/gitlab!71450

Add description of fuzz testing steps

See merge request gitlab-org/gitlab!70568

CTRT edits for the Notification emails doc

See merge request gitlab-org/gitlab!71651

Use allowlist of allowed attributes for imported models (part 3)

See merge request gitlab-org/gitlab!71296

Add HTTP Basic support for Alert authentication

See merge request gitlab-org/gitlab!70588

Improve merge train help text [RUN-AS-IF-FOSS]

See merge request gitlab-org/gitlab!71687

Docs: Note that after_script behavior is variable

See merge request gitlab-org/gitlab!71782

Add blob search aggregations for language

See merge request gitlab-org/gitlab!71937

VSA: Ensure selected projects are always visible

See merge request gitlab-org/gitlab!72115

Merge branch '339507-determine-if-manual-vulnerability-creation-mutation-should-respect-security-scanners-schema' into 'master'

Adjust Mutations::Vulnerabilities::Create to respect security scanner schema

See merge request gitlab-org/gitlab!69961

Convert gold to Ultimate in trial onboarding

See merge request gitlab-org/gitlab!72193

Deprecate promoted_issues metric

See merge request gitlab-org/gitlab!71604

Feature flag to rollout new CustomersDot Staging URL

See merge request gitlab-org/gitlab!71827

This change changes the behavior that is reponsible for filtering out
attributes when importing new project. We were cleaning attributes based
on list of excluded patterns and attributes. For ProtectedBranch,
ProtectedTag, ProtectedEnvironment and Releases we are changing that
behavior.

Changelog: changed

Enable project shared runners query change FF

See merge request gitlab-org/gitlab!71816

Remove direct setup of `shared_runners_seconds` in specs

See merge request gitlab-org/gitlab!71676

E2E: Add testcase link to group import spec

See merge request gitlab-org/gitlab!72196

Expose issue contacts via GraphQL

See merge request gitlab-org/gitlab!71889