Page Index Toggle Pages: 1 Send TopicPrint
Hot Topic (More than 5 Replies) Bug: BM removes trailing slashes, breaks bookmarks (Read 7474 times)
am2112
Junior Member
Offline
Posts: 5
Joined: May 30th, 2011
Re: Bug: BM removes trailing slashes, breaks bookmarks
Reply #7 - May 25th, 2012 at 8:39am
Print Post  
Aha!  This is funny.  I found the cocoabuilder thread first, which reminded me of a problem I had been facing with BookMacster and then dropped by this post.  Wink

I'm now trying out 1.11 and it works great!  Thanks!
  
Back to top
 
IP Logged
 
Jerry Krinock
Admin
Offline
Posts: 1666
Location: San Jose, California, USA
Joined: Sep 29th, 2008
Re: Bug: BM removes trailing slashes, breaks bookmarks
Reply #6 - May 23rd, 2012 at 12:14pm
Print Post  
am2112 wrote on May 23rd, 2012 at 8:05am:
I'm also the one who has been suffering from this problem, and would love to see this addressed.
BookMacster 1.11 has the issue fixed.  If you don't have BookMacster 1.11, launch BookMacster and click in the menu: BookMacster ▸ Check for Update.

am2112 wrote on May 23rd, 2012 at 8:05am:
There was a discussion recently on cocoabuilder.com, FYI.  Try googling for "nsurl path slash".
Indeed!  You'll notice that thread was started by a guy named Jerry Krinock  Wink

am2112 wrote on May 23rd, 2012 at 8:05am:
(I'm not allowed to post a URL yet)
Sorry, spammers have been going crazy lately.  People are apparently employed to read the CAPTCHA challenges and register for forums.  We'll have to maintain these restrictions until available wages in east Asia rise to the point where this spamming boards is no longer perceived as economical.
  

Jerry Krinock
Back to top
IP Logged
 
am2112
Junior Member
Offline
Posts: 5
Joined: May 30th, 2011
Re: Bug: BM removes trailing slashes, breaks bookmarks
Reply #5 - May 23rd, 2012 at 8:05am
Print Post  
I'm also the one who has been suffering from this problem, and would love to see this addressed.

There was a discussion recently on cocoabuilder.com, FYI.
Try googling for "nsurl path slash".

(I'm not allowed to post a URL yet)
  
Back to top
 
IP Logged
 
Jerry Krinock
Admin
Offline
Posts: 1666
Location: San Jose, California, USA
Joined: Sep 29th, 2008
Re: Bug: BM removes trailing slashes, breaks bookmarks
Reply #4 - Apr 10th, 2012 at 2:44am
Print Post  
You're correct, deanishe.  Although rare, URLs may have trailing slashes on their path components.  BookMacster was trimming these as part of its normalization, thanks to Apple's URL path-extraction method (-[NSURL path]) which, for some reason, trims trailing slashes.  I have no idea what they were thinking back 12+ years ago when they wrote that.  Anyhow, we found another method which behaves properly, so this is fixed in the next version of BookMacster.
  

Jerry Krinock
Back to top
IP Logged
 
Jerry Krinock
Admin
Offline
Posts: 1666
Location: San Jose, California, USA
Joined: Sep 29th, 2008
Re: Bug: BM removes trailing slashes, breaks bookmarks
Reply #3 - Apr 7th, 2012 at 1:18pm
Print Post  
Sorry, looks like my response was a little too speedy.

OK, I see the problem.  I'll get back to you on this.  Thanks for the report.
  

Jerry Krinock
Back to top
IP Logged
 
deanishe
New Member
Offline
Posts: 2
Joined: Apr 7th, 2012
Re: Bug: BM removes trailing slashes, breaks bookmarks
Reply #2 - Apr 7th, 2012 at 1:08pm
Print Post  
It's not adding them that's the (well, my) problem, it's removing them.

Thanks for the speedy response.
  
Back to top
 
IP Logged
 
Jerry Krinock
Admin
Offline
Posts: 1666
Location: San Jose, California, USA
Joined: Sep 29th, 2008
Re: Bug: BM removes trailing slashes, breaks bookmarks
Reply #1 - Apr 7th, 2012 at 12:53pm
Print Post  
OK, we'll reconsider if we can make this an option.  Originally, BookMacster did not add trailing slashes, but that caused syncing problems with browsers that insisted on adding the trailing slash.  I'll re-study the issue.
  

Jerry Krinock
Back to top
IP Logged
 
deanishe
New Member
Offline
Posts: 2
Joined: Apr 7th, 2012
Bug: BM removes trailing slashes, breaks bookmarks
Apr 7th, 2012 at 12:44pm
Print Post  
Hi,

I've noticed BookMacster will delete trailing slashes from URLs, e.g. hxxp://example.com/blah/ becomes hxxp://example.com/blah when BookMacster imports it, and stays that way when it exports it. (I had to use hxxp because the forum won't let me post URLs yet, even ones that aren't real.)

This behaviour is breaking some of my bookmarks, and there appears to be no option to disable it.

I realise that truncating trailing slashes generally works (because most servers are smart enough to do the right thing when fed the wrong URL) and is helpful in avoiding duplicates, but BM should not be silently altering bookmarked URLs. Period.

The trailing slash is there for a reason: because that is the correct URL.

Please stop altering bookmark URLs. It breaks them.
  
Back to top
 
IP Logged
 
Page Index Toggle Pages: 1
Send TopicPrint