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

Rewrite /cdns/configs to Go #3849

Closed
3 of 17 tasks
ocket8888 opened this issue Aug 7, 2019 · 0 comments · Fixed by #4353
Closed
3 of 17 tasks

Rewrite /cdns/configs to Go #3849

ocket8888 opened this issue Aug 7, 2019 · 0 comments · Fixed by #4353
Labels
tech debt rework due to choosing easy/limited solution Traffic Ops related to Traffic Ops
Milestone

Comments

@ocket8888
Copy link
Contributor

ocket8888 commented Aug 7, 2019

I'm submitting a ...

  • bug report
  • new feature / enhancement request
  • improvement request (usability, performance, tech debt, etc.)
  • other

Traffic Control components affected ...

  • CDN in a Box
  • Documentation (possibly)
  • Grove
  • Traffic Control Client
  • Traffic Monitor
  • Traffic Ops
  • Traffic Ops ORT
  • Traffic Portal
  • Traffic Router
  • Traffic Stats
  • Traffic Vault
  • unknown

Current behavior:

/cdns/configs is handled by Perl

Expected / new behavior:

/cdns/configs should be handled by Go.

Minimum API Version

1.2 - This is the actual problem here. The current Go implementation has a minimum supported version of 1.1

Request Methods

  • GET

Minimal reproduction of the problem with instructions:

N/A

@ocket8888 ocket8888 added Traffic Ops related to Traffic Ops tech debt rework due to choosing easy/limited solution labels Aug 7, 2019
@ocket8888 ocket8888 added this to the Go Rewrite milestone Aug 12, 2019
@mhoppa mhoppa mentioned this issue Jan 29, 2020
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech debt rework due to choosing easy/limited solution Traffic Ops related to Traffic Ops
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant