Skip to content
This repository has been archived by the owner on Jun 16, 2022. It is now read-only.

Not allowed root "/" in old url #79

Closed
yonatancohen opened this issue May 21, 2015 · 14 comments
Closed

Not allowed root "/" in old url #79

yonatancohen opened this issue May 21, 2015 · 14 comments

Comments

@yonatancohen
Copy link

Hi, I want to define the next rule :
capture

However i get an error that the rule is invalid... Can't i use the "/" in old url?
when I look in the DB it's save as "NULL", but if i change the value to "/" - all works fine..

Anyway for quick fixing?

@kipusoep
Copy link
Owner

The reason is because the smallest URL is being stored; this means without leading and trailing slash. That's why NULL is stored in the database.

@yonatancohen
Copy link
Author

Anyway to override that option?

I have the following urls :

Can i do that using the "Url Tracker"?

Thanks !!!!!

‫בתאריך יום ה׳, 21 במאי 2015 ב-17:34 מאת ‪Stefan Kip‬‏ <‪
[email protected]‬‏>:‬

The reason is because the smallest URL is being stored; this means without
leading and trailing slash. That's why NULL is stored in the database.


Reply to this email directly or view it on GitHub
#79 (comment).

@kipusoep
Copy link
Owner

Not via the UI, if it works via SQL you're lucky :-)

@yonatancohen
Copy link
Author

Ok, so i'll download the code and try to add it myself.

Thanks,

@kipusoep
Copy link
Owner

I'll try to fix this soonish :-)

@yonatancohen
Copy link
Author

I will really appreciate it!!
On Fri, May 22, 2015 at 17:27 Stefan Kip [email protected] wrote:

I'll try to fix this soonish :-)


Reply to this email directly or view it on GitHub
#79 (comment).

@yonatancohen
Copy link
Author

Hi, I wonder if you go into it?

Thanks,

‫בתאריך יום ו׳, 22 במאי 2015 ב-17:29 מאת ‪Yoni Cohen‬‏ <‪
[email protected]‬‏>:‬

I will really appreciate it!!
On Fri, May 22, 2015 at 17:27 Stefan Kip [email protected] wrote:

I'll try to fix this soonish :-)


Reply to this email directly or view it on GitHub
#79 (comment)
.

@kipusoep
Copy link
Owner

Fixed in 2fad9b7, not released yet though.

@yonatancohen
Copy link
Author

Ok.. So first thanks a lot !
Second, when do you think you will release it?
On Tue, May 26, 2015 at 23:32 Stefan Kip [email protected] wrote:

Fixed in 2fad9b7
2fad9b7,
not released yet though.


Reply to this email directly or view it on GitHub
#79 (comment).

@kipusoep
Copy link
Owner

When there's a bit more to release. You can download the solution and create a build for yourself though, no sweat.

@yonatancohen
Copy link
Author

Hi, now it's allow to add "/" at root - but doesn't do the redirect itself.

‫בתאריך יום ג׳, 26 במאי 2015 ב-23:34 מאת ‪Stefan Kip‬‏ <‪
[email protected]‬‏>:‬

When there's a bit more to release. You can download the solution and
create a build for yourself though, no sweat.


Reply to this email directly or view it on GitHub
#79 (comment).

@kipusoep
Copy link
Owner

It works for me though, altough I tried by setting no root node (root node set to the first option '/').
Does it work if you select no root node (first option)?

@yonatancohen
Copy link
Author

Both doesn't seem to work,

I set the root (in combo) to "/",
Enter some query "Id=123",
and the redirect path to "http://www.google.com"

‫בתאריך יום ה׳, 28 במאי 2015 ב-15:03 מאת ‪Stefan Kip‬‏ <‪
[email protected]‬‏>:‬

It works for me though, altough I tried by setting no root node (root node
set to the first option '/').
Does it work if you select no root node (first option)?


Reply to this email directly or view it on GitHub
#79 (comment).

@kipusoep
Copy link
Owner

Then I'm not sure what's going on to be honest...

kipusoep added a commit that referenced this issue Jul 9, 2015
[BugFix] AdvanceView is throwing an exception when having a single domain configured in umbraco (#86)
[BugFix] Fixed possible Null Reference exception while looping through the forced redirects collection
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants