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.

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

Removes v0, redirect downloads #355

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Commits on Aug 14, 2018

  1. Removes v0, redirect downloads

    Removes all v0 files. these endpoints now return 410 (gone), except for
    the download action, which rewrites the url to return a redirect to the v1 api endpoint.
    
    We may consider writing an nginx directive to handle this as it would be
    far faster.
    NyanHelsing committed Aug 14, 2018
    Configuration menu
    Copy the full SHA
    f23c6b5 View commit details
    Browse the repository at this point in the history
  2. Fix missing slash, unused imports

    Removes some unused imports and adds in a slash that was missiong from
    the url being redirected to.
    NyanHelsing committed Aug 14, 2018
    Configuration menu
    Copy the full SHA
    14de20b View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    bb6cdca View commit details
    Browse the repository at this point in the history
  4. Remove unused import

    NyanHelsing committed Aug 14, 2018
    Configuration menu
    Copy the full SHA
    9fe375d View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    9f6054f View commit details
    Browse the repository at this point in the history

Commits on Aug 16, 2018

  1. Remove AuthHandler fetch

    Only v0 uses the fetch method on the auth handler. v1 uses the get
    method. This removes the fetch method and the tests for it.
    NyanHelsing committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    4df6c22 View commit details
    Browse the repository at this point in the history
  2. Remove validate path, and rename validate_v1_path

    validate_path is only used by v0, so by removing v0, we can remove
    validate_path as well. validate_v1_path also gets renamed validate_path
    to cut down any confusion about the naming
    NyanHelsing committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    1ce86b9 View commit details
    Browse the repository at this point in the history
  3. Update tests

    Some tests only tested validate_path, which has been removed, so these
    tests were removed. Other tests needed to be updated so that they were
    mocked correctly because validate_v1_path (now `validate_path`) makes
    network requests.
    NyanHelsing committed Aug 16, 2018
    Configuration menu
    Copy the full SHA
    7ad143c View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    3286578 View commit details
    Browse the repository at this point in the history