Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

GET /api/4.0/servers?profileId={id} does not include all profiles #6834

Closed

Description

@ericholguin

This Bug Report affects these Traffic Control components:

  • Traffic Ops

Current behavior:

When making a GET request /api/4.0/servers?profileId={id} where id belongs to a profile that is not index 0 in the profileNames array the response will not display that server.

For example if there exists a server with the profileNames:

{
...
 "profileNames": [
                "EDGE_TIER_ATS_CACHE", "MID_TIER_ATS_CACHE"
            ],
...
}

Where the id of the EDGE_TIER_ATS_CACHE profile is 13 and the id of MID_TIER_ATS_CACHE is 12.

If we perform a GET request: /api/4.0/servers?profileId=13 we will see the server above returned in the response, however
if we perform a GET request: /api/4.0/servers?profileId=12 (the following profile - at index 1 in the array) that server will not be included in the response.

Expected behavior:

Should account for all profiles, including profiles not at index 0 in the profileNames array

Steps to reproduce:

Have a server with multiple profiles, make a GET request /api/4.0/servers?profileId={id} where id belongs to the profile NOT at index 0, you will not see the server included in the response

Activity

  1. added
    bugsomething isn't working as intended
    Traffic Opsrelated to Traffic Ops
    low impactaffects only a small portion of a CDN, and cannot itself break one
    on May 16, 2022
  2. ocket8888 commented on May 16, 2022

    @ocket8888
    Contributor

    profileId should be deprecated as a query string parameter, since Profiles are not represented by their IDs on servers as of APIv4. Should be profileName= instead in APIv4 and later.

  3. mitchell852 commented on May 17, 2022

    @mitchell852
    Member

    Should be profileName= instead in APIv4 and later

    yes, i would agree with this. no need for a query param of profileId in v4+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Traffic Opsrelated to Traffic Opsbugsomething isn't working as intendedlow impactaffects only a small portion of a CDN, and cannot itself break one

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions