[PR #20] [MERGED] Get rid of Iris and use julienschmidt/httprouter instead #251

New issue

Closed

opened 2026-03-13 16:13:35 +03:00 by kerem · 0 comments

kerem commented

2026-03-13 16:13:35 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/acme-dns/acme-dns/pull/20
Author: @joohoi
Created: 11/14/2017
Status: ✅ Merged
Merged: 11/14/2017
Merged by: @joohoi

Base: master ← Head: httprouter

📝 Commits (5)

40539cf Replace iris with httprouter
e183243 Linter fixes
4a22f32 Finalize iris removal
d618c95 Vendor dependencies for reproducable builds
1c401e7 Api tests are back

📊 Changes

1541 files changed (+668170 additions, -260 deletions)

View changed files

📝 .gitignore (+0 -2)
📝 Gopkg.lock (+11 -155)
📝 Gopkg.toml (+35 -4)
📝 acmetxt.go (+4 -0)
📝 api.go (+50 -85)
➕ api_test.go (+310 -0)
➕ auth.go (+87 -0)
📝 main.go (+50 -13)
📝 types.go (+1 -1)
📝 util.go (+5 -0)
➕ vendor/github.com/BurntSushi/toml/.gitignore (+5 -0)
➕ vendor/github.com/BurntSushi/toml/.travis.yml (+15 -0)
➕ vendor/github.com/BurntSushi/toml/COMPATIBLE (+3 -0)
➕ vendor/github.com/BurntSushi/toml/COPYING (+14 -0)
➕ vendor/github.com/BurntSushi/toml/Makefile (+19 -0)
➕ vendor/github.com/BurntSushi/toml/README.md (+218 -0)
➕ vendor/github.com/BurntSushi/toml/_examples/example.go (+61 -0)
➕ vendor/github.com/BurntSushi/toml/_examples/example.toml (+35 -0)
➕ vendor/github.com/BurntSushi/toml/_examples/hard.toml (+22 -0)
➕ vendor/github.com/BurntSushi/toml/_examples/implicit.toml (+4 -0)

...and 80 more files

📄 Description

Iris can't keep the API stable, and while looking into it, found out a lot of drama behind the scenes too. This change will also rid us of hefty dependency stack brought in by iris.

Brought back the HTTP api tests as well!

Fixes #16
Fixes #11

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/acme-dns/acme-dns/pull/20 **Author:** [@joohoi](https://github.com/joohoi) **Created:** 11/14/2017 **Status:** ✅ Merged **Merged:** 11/14/2017 **Merged by:** [@joohoi](https://github.com/joohoi) **Base:** `master` ← **Head:** `httprouter` --- ### 📝 Commits (5) - [`40539cf`](https://github.com/acme-dns/acme-dns/commit/40539cf335b0dc31a8052dd602e43d94e3b67f6d) Replace iris with httprouter - [`e183243`](https://github.com/acme-dns/acme-dns/commit/e183243ed962b69d7bdef386059a51a1da1c94bb) Linter fixes - [`4a22f32`](https://github.com/acme-dns/acme-dns/commit/4a22f320ab21d0fcab4eab1981eea95a28be9b92) Finalize iris removal - [`d618c95`](https://github.com/acme-dns/acme-dns/commit/d618c95c4334b9ce1fe233402223557e588bec8a) Vendor dependencies for reproducable builds - [`1c401e7`](https://github.com/acme-dns/acme-dns/commit/1c401e7df2a584a3499f1a251fe832f9c893fdd7) Api tests are back ### 📊 Changes **1541 files changed** (+668170 additions, -260 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+0 -2) 📝 `Gopkg.lock` (+11 -155) 📝 `Gopkg.toml` (+35 -4) 📝 `acmetxt.go` (+4 -0) 📝 `api.go` (+50 -85) ➕ `api_test.go` (+310 -0) ➕ `auth.go` (+87 -0) 📝 `main.go` (+50 -13) 📝 `types.go` (+1 -1) 📝 `util.go` (+5 -0) ➕ `vendor/github.com/BurntSushi/toml/.gitignore` (+5 -0) ➕ `vendor/github.com/BurntSushi/toml/.travis.yml` (+15 -0) ➕ `vendor/github.com/BurntSushi/toml/COMPATIBLE` (+3 -0) ➕ `vendor/github.com/BurntSushi/toml/COPYING` (+14 -0) ➕ `vendor/github.com/BurntSushi/toml/Makefile` (+19 -0) ➕ `vendor/github.com/BurntSushi/toml/README.md` (+218 -0) ➕ `vendor/github.com/BurntSushi/toml/_examples/example.go` (+61 -0) ➕ `vendor/github.com/BurntSushi/toml/_examples/example.toml` (+35 -0) ➕ `vendor/github.com/BurntSushi/toml/_examples/hard.toml` (+22 -0) ➕ `vendor/github.com/BurntSushi/toml/_examples/implicit.toml` (+4 -0) _...and 80 more files_ </details> ### 📄 Description Iris can't keep the API stable, and while looking into it, found out a lot of drama behind the scenes too. This change will also rid us of hefty dependency stack brought in by iris. Brought back the HTTP api tests as well! Fixes #16 Fixes #11 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>