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

urlshortener does not work in master #280

Closed
rogierm opened this issue Feb 7, 2018 · 4 comments
Closed

urlshortener does not work in master #280

rogierm opened this issue Feb 7, 2018 · 4 comments

Comments

@rogierm
Copy link

rogierm commented Feb 7, 2018

Steps to reproduce

  1. git clone master
  2. update conf.php to enable urlshortener
  3. There is no Shorten URL button.

What happens

No Shorten URL button appears when you enable urlshortener in conf.php

What should happen

The Shorten URL button should appear

Additional information

When I checkout tags/1.1.1 the Shorten URL button works as expected.

Basic information

Server address:

Server OS: CentOS 7

Webserver: Nginx PHP-FPM

Browser:

PrivateBin version: master does not work

I can reproduce this issue on https://privatebin.net: Yes / No

@rugk rugk added the bug label Feb 7, 2018
@sprnza
Copy link

sprnza commented Feb 16, 2018

I was struggling with it for 2 hours! I thought something's wrong with my url-shorteners!! I don't have the shorten button even with 1.1.1 though. Cache problems. The button is there in 1.1.1

@rugk
Copy link
Member

rugk commented Feb 16, 2018

But you can confirm it does not work in master?

@sprnza
Copy link

sprnza commented Feb 17, 2018

Yes, no button in master.

@elrido
Copy link
Contributor

elrido commented Feb 20, 2018

@rugk I did test it on my instance and could also replicate it. The HTML with the "data-shortener"-property containing the configured URL gets generated, but when sending the paste, the JS removes the "shortenbutton" ID, regardless of the presence of the "data-shortener"-property being there or not.

I assume this is a regression of the JS refactoring. Will try to provide you a unit test case over the weekend. Would be great if you could look into this.

rugk added a commit that referenced this issue Feb 21, 2018
Move URL shortener out of #pastelink as that is overwritten by the JS.

Fixes #280
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

No branches or pull requests

4 participants