search

romainletendart / geecxx

IRC bot written in C++11
BSD 2-Clause "Simplified" License
5 stars 0 forks source link

Shorten URLs when showing page's title #8

Closed jsilvestre closed 9 years ago

jsilvestre commented 9 years ago

So we don't have big URLs twice in the chat log.

Thank you very much!

hamadr commented 9 years ago

I don't agree with this feature. There is a major issue with url shorteners : privacy. Maybe we should only do it if the user said so. What do you think @romainletendart?

vaz-ar commented 9 years ago

Why not integrate an url shortener in the bot ?

jsilvestre commented 9 years ago

@vaz-ar nailed it :)

romainletendart commented 9 years ago

URL shortening seems a good idea but I'm not really willing to make it part of the bot's features. This would mean the bot should provide its own web server, have its own domain name, etc.

Instead, I would prefer to specify at runtime what URL shortening service to use (if any).

romainletendart commented 9 years ago

Do you consider the "title-only" display sufficient to close this issue?

jsilvestre commented 9 years ago

Yes sir.

jsilvestre commented 9 years ago

Thank you again <3