Commit Graph

176 Commits

Author SHA1 Message Date
Shane Smith-Sahnow
ad4ce0eb16 updating logic in route matcher, cleaner and saner (#235) 2017-02-27 19:44:49 -08:00
Kamil Kisiel
999ef73f5d Merge pull request #232 from DavidJFelix/patch-1
Add sourcegraph badge
2017-02-23 12:25:54 -08:00
David J. Felix
89d16fe9a0 Add sourcegraph badge
Fixes #228
2017-02-23 14:57:28 -05:00
Kamil Kisiel
94e7d24fd2 Add Go 1.8 to .travis.yml 2017-02-17 11:26:16 -08:00
Adam Eijdenberg
392c28fe23 [bugfix] fail fast if regex is incorrectly specified using capturing groups. (#218) v1.3.0 2017-01-18 05:43:44 -08:00
Shawn Smith
cafdb65e9e [docs] Add route listing example to README
* Update README.md

* Update README.md
2017-01-18 05:43:23 -08:00
Kamil Kisiel
b12896167c Merge pull request #199 from wirehead/minor-doc-tweek
Clarify how route variables work. (#151)
v1.2.0
2017-01-16 23:01:07 -08:00
Kamil Kisiel
34dda716af Merge pull request #215 from ShaneSaww/fix_for_subroutes_with_pathPrefix
Adding in a check for routes with just /
2017-01-16 22:41:11 -08:00
ShaneSaww
b9ff34f617 Adding some extra tests, to hit all the use cases 2017-01-16 21:11:35 -08:00
ShaneSaww
293ebe1493 Adding in a check for routes with just / 2017-01-15 21:47:00 -08:00
Ken Wronkiewicz
910dd3aa31 Remove unnecessary line from example
Review comment: setting the header complicates things.
2016-10-03 15:30:11 -07:00
Ken Wronkiewicz
239e05fe53 Clarify how route variables work. (#151)
(this confused the heck out of me while trying to use Gorilla)

* Changed `request` to be just `r`, like the other handlers.
* Created complete wrapper function instead of just 2 lines.
2016-09-30 10:54:10 -07:00
Kamil Kisiel
757bef944d Merge pull request #196 from olt/doc-non-capture-groups
document non-capturing groups (closes #143)
2016-09-20 16:08:13 -07:00
Oliver Tonnhofer
14f5df0128 document non-capturing groups (closes #143) 2016-09-20 16:06:56 +02:00
Kush Mansingh
0a192a1931 Add useEncodedPath option to router and routes (#190)
- Resolves a breaking change in #184
2016-09-02 08:33:43 -07:00
Richard Musiol
0b13a92220 Simplify extractVars, fixes edge cases. (#185)
Also make sure all regexp groups in tests are non-capturing.
2016-08-27 09:50:34 -07:00
Aaron Taylor
34bf6dc9fa make the getPath method safer, fixing panics within App Engine (#189) 2016-08-24 16:34:02 -07:00
Kush Mansingh
674ef1c280 Add mechanism to route based on the escaped path (#184)
* Add mechanism to route based on the escaped path, correct request mocking in tests

* Remove unneccessary regex matching, substitute with string slicing

* Add test case and handling for requests with no host/scheme
2016-08-24 06:45:17 -07:00
Kamil Kisiel
cf79e51a62 .travis.yml: add go1.7 2016-08-16 11:46:30 -07:00
Matt Silverlock
7804150971 [docs] Add logo to README. (#180) 2016-08-01 10:00:46 -07:00
Matt Silverlock
3b15f12d15 [docs] Add static file example to README; doc.go. (#179) 2016-08-01 08:51:14 -07:00
Kamil Kisiel
327d4b684c Clean up some naming in mux_test.go 2016-07-30 16:45:59 -07:00
Matt Silverlock
a49d62f618 [bugfix] Fix error handling in Router.Walk (#177) 2016-07-26 18:54:17 -07:00
Martin Hamrle
cf57124f1d Fix error handling in Router.Walk
In old version error was not returned properly, In one walkFn call
error was checked only for SkipRouter but not for nil.
2016-07-25 18:05:24 +02:00
Matt Silverlock
d391bea311 [docs] README typo (#175) 2016-07-18 08:11:58 -07:00
delong
198f95d824 fix README.md 2016-07-18 13:28:55 +08:00
Matt Silverlock
9fa818a44c [ci] Update .travis-ci to skip install block. (#170)
Ref 12a13f34e4 (commitcomment-17371551)
2016-06-05 16:35:21 -07:00
Kamil Kisiel
e84fac997f Merge pull request #169 from ejholmes/go1.7-context
Store vars and route in context.Context when go1.7+ is used
2016-06-04 09:35:56 -07:00
Eric J. Holmes
fdfca9f917 Support native context.Context when go1.7 is used. 2016-06-04 15:38:01 +07:00
Eric J. Holmes
5dd56998c2 Add failing context.Context test for go1.7. 2016-06-04 15:21:55 +07:00
Matt Silverlock
bd09be08ed [docs] Log ListenAndServe's err response in example 2016-05-25 07:09:13 -07:00
Mukhtar Haji
0d753bb8ca Log ListenAndServe's err response. 2016-05-25 14:44:10 +01:00
Matt Silverlock
9c19ed558d [feature] Add SkipClean option
- SkipClean skips cleaning routes (or a route) - e.g. when true, if the route path is "/path//to", it will remain with the double slash. This is helpful if you have a route like: /fetch/http://xkcd.com/534/
2016-05-02 10:56:24 -07:00
Jingwen Owen Ou
12a13f34e4 go vet is part of Go now 2016-05-02 10:30:24 -07:00
Jingwen Owen Ou
05d9d908e7 Add SkipClean() to *Route 2016-05-02 10:24:47 -07:00
Jingwen Owen Ou
9935257381 Add test for SkipClean 2016-05-02 10:24:18 -07:00
Jingwen Owen Ou
a41434aac3 Clarify more of SkipClean 2016-05-02 10:24:09 -07:00
Matt Silverlock
0eeaf8392f Merge pull request #158 from raninho/master
[tests] old_test: const DefaultRemoteAddr is unused.
v1.1
2016-03-17 14:34:30 -07:00
Raniere Fernandes de Medeiros
8e05c69b5a old_test: const DefaultRemoteAddr is unused. 2016-03-17 18:23:30 -03:00
Matt Silverlock
acf3be1b33 Merge pull request #155 from jprobinson/master
[bugfix] Allow case insensitive regexp groups (fix regression in #144)
2016-03-07 14:01:44 -08:00
JP Robinson
65c2651643 fixing regexp changes from PR #144 2016-03-07 16:38:03 -05:00
Jingwen Owen Ou
786d36e5ab go fmt 2016-03-07 11:43:49 -08:00
Dave Newman
8ac5cf967f Add SkipClean option
By default paths are run through the cleanPath method which prevents
using fancier paths like /fetch/http://xkcd.com/534

This adds a SkipClean option so that this path isn't redirected to
/fetch/http/xkcd.com/534
2016-03-07 11:41:38 -08:00
Matt Silverlock
147a95f5e3 Merge pull request #104 from Digitalxero/method-to-get-url-template
[feature] GetHostTemplate and GetPathTemplate return the template used to build the route.
2016-03-05 05:56:38 -08:00
Dj Gilcrease
f7ac7251e3 Update doc strings based on feedback 2016-02-29 07:54:15 -08:00
Matt Silverlock
c9e326e2bd Merge pull request #150 from gorilla/elithrar/go-lint
[docs] Satisfied golint.

- SkipRouter should also be ErrRouterSkipped (or similar) but a change would break the public API.
2016-02-28 19:40:56 -08:00
Matt Silverlock
02c98b3f73 [docs] Satisfied golint.
- SkipRouter should also be ErrRouterSkipped (or similar) but a change would
  break the public API.
2016-02-28 19:32:22 -08:00
Dj Gilcrease
f84ab9ab62 Fix go fmt issues 2016-02-28 14:46:18 -08:00
Dj Gilcrease
0d60c4bfeb Add tests for GetPathTemplate. Added GetHostTemplate and associated tests as well 2016-02-28 14:42:09 -08:00
Dj Gilcrease
16507e2c47 Merge branch 'master' into method-to-get-url-template 2016-02-28 13:16:25 -08:00