Allow custom callback urls about scribejava HOT 6 CLOSED

Shay,

Working on it right now

from scribejava.

Thanks!
If you're interested, what I did, is just made sure it's a valid URI rather than URL, and that fixed everything I needed.
Great package by the way! :)

Shay

from scribejava.

Cool.

What I changed is this:

http://gist.github.com/616146

Let me know what you think about it

from scribejava.

Looks cool...
I did

static boolean isUrl(String url)
{
try {
new URI(url);
} catch (URISyntaxException e) {
return false;
}
return true;
// return url.startsWith("http://") || url.startsWith("https://");
}

I'm not sure performance wise what is better, regex or using try/catch, but I think both will work equally good, and since it's only once per service, it's not that bad anyway.

Thanks again

from scribejava.

Cool.

I kinda have a weakness for regexes, and don't like the try catch blocks a lot so I think I'll leave as it is for now...

You have the fix on 1.0.4 released just now.

sorry for the delay I've been working my ass of lately :)

from scribejava.

I know your point of view... I love regexes as well...
Anyway, thanks! I'll let you know if I find more things in the package....

btw - are you thinking about integrating with OAuth 2.0 ? (What Facebook is using...), it's almost the same as OAuth 1, but with some minor changes

from scribejava.