Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

extract bluecat api code into its own package #2696

Merged
merged 8 commits into from
Apr 12, 2022
Merged

extract bluecat api code into its own package #2696

merged 8 commits into from
Apr 12, 2022

Commits on Apr 8, 2022

  1. provider/bluecat: migrate client code to separate pkg 

    Extract all Bluecat Gateway Client code to separate package
    @vinny-sabatini
    vinny-sabatini committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    83f062f View commit details
    Browse the repository at this point in the history

  2. provider/bluecat: improve api error handling 

    Check status codes when API calls are made for gets and deletes. All of
the creates were done slightly different and will be updated in a
separate commit.
    @vinny-sabatini
    vinny-sabatini committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    17d4371 View commit details
    Browse the repository at this point in the history

  3. provider/bluecat: handle http errors for create in API

    @vinny-sabatini
    vinny-sabatini committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    5296a15 View commit details
    Browse the repository at this point in the history

  4. provider/bluecat: fix txt pre/suffix bug 

    The prefix and suffix for the txt record was not being taken into
account when trying to find an owner for a cname and host record.

In addition, CName records have to be unique values, therefore txt records need to
have either a prefix or suffix included. This fixes the "owner" logic to
ensure when checking a corresponding txt recod against a cname record
that the prefix or suffix is used when comparing records.

Also, correct the response codes for the DELETE calls to Bluecat Gateway
    @vinny-sabatini
    vinny-sabatini committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    cbe1969 View commit details
    Browse the repository at this point in the history

  5. provider/bluecat: do not deploy dns on dry-run

    @vinny-sabatini
    vinny-sabatini committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    5851d43 View commit details
    Browse the repository at this point in the history

  6. provider/bluecat: extract http requests into function 

    Consolidating the HTTP requests into a function will help ensure
consistency when making HTTP requests to the Bluecat gateway, and in a
future commit will make mocking these requests easier
    @vinny-sabatini
    vinny-sabatini committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    3b68190 View commit details
    Browse the repository at this point in the history

  7. provider/bluecat: minor test cleanup 

    * Move tests to appropriate package
* Add test for SplitProperties function
* Remove unused helper function
* Use table driven tests
    @vinny-sabatini
    vinny-sabatini committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    0d16c66 View commit details
    Browse the repository at this point in the history

  8. provider/bluecat: first set of tests for api

    @vinny-sabatini
    vinny-sabatini committed Apr 8, 2022
    Configuration menu
    Copy the full SHA
    d4927b2 View commit details
    Browse the repository at this point in the history