Skip to content
This repository has been archived by the owner on Jan 24, 2019. It is now read-only.

Add option to set custom footer text #256

Closed
wants to merge 1 commit into from

Conversation

emmaly
Copy link

@emmaly emmaly commented May 25, 2016

Intended to resolve #166. Adds an option (in all the usual spots) called "footer" that allows for overriding the existing footer text. It's a static value. If you want the footer to go away altogether, just set it to " " (single space, not empty string). If it's set to empty string (default), it'll do the usual version/link footer. I expect that it's simple plaintext and any HTML would be escaped by the template.

@emmaly
Copy link
Author

emmaly commented May 29, 2016

I don't believe the CI test failures are due to my patch. Other PRs near the same time were giving the same error, and tests run fine on my machines. Please complain to me if I'm wrong about that. I'm using this patch in production with great success.

@rremer
Copy link
Contributor

rremer commented Jun 11, 2016

@dustywilson fetch upstream and push again, I believe you (like me) were building with a recent version of golang which travis was not. Some upstream deps were no longer available for those ancient versions of golang. The travis build works now (see #249 ), so I suspect your build will pass once you rebase.

@jehiah jehiah closed this in 57f82ed Jun 19, 2016
ryodocx pushed a commit to ryodocx/oauth2_proxy that referenced this pull request Mar 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

option to hide version number
2 participants