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

[backport] PR #6581 to 4.x #6596

Merged
merged 1 commit into from
Mar 22, 2016
Merged

[backport] PR #6581 to 4.x #6596

merged 1 commit into from
Mar 22, 2016

Conversation

elastic-jasper
Copy link
Contributor

Backport PR #6581 to 4.x

[Commit 1]
short url: ensure absolute path isn't persisted

[Commit 2]
[short url] use url.format when creating /goto link

[Commit 3]
[short url] Add tests for query strings, no hashes

[Commit 4]
[short url] Cleanup

[Commit 5]
[short url] Add server tests

[Commit 1]
short url: ensure absolute path isn't persisted

Original sha: a8c1305
Authored by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-18T20:02:53Z
Committed by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T15:57:16Z

[Commit 2]
[short url] use url.format when creating /goto link

Original sha: 2a71673
Authored by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T14:09:04Z
Committed by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T15:57:16Z

[Commit 3]
[short url] Add tests for query strings, no hashes

Original sha: 690a140
Authored by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T14:42:10Z
Committed by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T15:57:16Z

[Commit 4]
[short url] Cleanup

Original sha: 49d297f
Authored by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T15:09:35Z
Committed by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T15:57:16Z

[Commit 5]
[short url] Add server tests

Original sha: a612a0e
Authored by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T15:30:12Z
Committed by Jonathan Budzenski <jbudz@users.noreply.github.com> on 2016-03-21T18:12:25Z
@epixa epixa force-pushed the jasper/backport/6581-4.x-issues/6127 branch from 08e39de to f8a872e Compare March 22, 2016 19:52
@epixa epixa assigned jbudz and unassigned epixa Mar 22, 2016
@epixa
Copy link
Contributor

epixa commented Mar 22, 2016

@jbudz Since this had conflicts, can you give this a once over before I merge it? No need to look at the 4.5 and 4.5.0, since they had the same conflicts.

@jbudz
Copy link
Member

jbudz commented Mar 22, 2016

LGTM

@jbudz jbudz assigned epixa and unassigned jbudz Mar 22, 2016
epixa added a commit that referenced this pull request Mar 22, 2016
@epixa epixa merged commit 3ed5902 into 4.x Mar 22, 2016
@epixa epixa deleted the jasper/backport/6581-4.x-issues/6127 branch March 22, 2016 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants