Skip to content

testserver: rename GetHTTPClient to GetUnauthenticatedHTTPClient #76241

@knz

Description

@knz

Suggested by @dhartunian

the problem to solve is that it's currently just too easy to create a test for a HTTP API without authentication, which does not invite the implementer to care about authn (nor authorization).

We can nudge this in the right direction by renaming the "simple" method to a longer name with a discouragement in the comment; then renaming the admin method to a "simple" name so that it's easier to find.

Jira issue: CRDB-13044

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-testingTesting tools and infrastructureC-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.E-easyEasy issue to tackle, requires little or no CockroachDB experienceE-starterMight be suitable for a starter project for new employees or team members.T-server-and-securityDB Server & Securitygood first issue

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions