[SOLVED] Issue with some plugins in ACP v0.5.2

Alex

I am wondering if it's a general issue or if it's only me. @julian is it working fine for you?

julian

Hi @Alex, you'll have to wait for @psychobunny to respond, as it is his plugin

We use Ghost as our blogging platform, and it is working fine.

Alex

Hey @julian thanks for your quick reply. As it's happening with all the SSO plugins like the Facebook one you made, I thought you would maybe have got an idea

Alex

@julian I found the problem, it's not the ACP. The plugins I mentioned "Blog Comments, Facebook SSO, Twitter SSO, Steam SSO, Google SSO and Emailer Local" uses this in their admin template:

require(['forum/admin/settings'], function(Settings) {

I changed it to:

require(['admin/settings'], function(Settings) {

and now it's working fine, it retrieves the info from the db and the save buttons works.

julian

@psychobunny was the base root for require.js changed to no longer point to the modules folder?

peter

@Alex

Glad I bumped into your post. Can you share where you are making this modification so that I can test it on my end? I'm hoping it will resolve the crashing issue with blog comments. I was reinstalling so many things thinking I misconfigured wordpress and nodebb somehow and kept trying to find where the admin error was coming from. It was really frustrating.

Alex

Hey @peter, unfortunately it didn't fix the crashing issue when publishing an article from wordpress to nodebb. Each time I press the button on wordpress it still crashes and get this in the console:

Error: Can't set headers after they are sent.
at ServerResponse.OutgoingMessage.setHeader (http.js:689:11)
at ServerResponse.header (/home/www/nodebb/node_modules/express/lib/response.js:662:10)
at ServerResponse.res.location (/home/www/nodebb/node_modules/express/lib/response.js:776:8)
at ServerResponse.redirect (/home/www/nodebb/node_modules/express/lib/response.js:815:8)
at /home/www/nodebb/node_modules/nodebb-plugin-blog-comments/library.js:166:10
at /home/www/nodebb/src/topics/create.js:164:5
at fn (/home/www/nodebb/node_modules/async/lib/async.js:641:34)
at Object._onImmediate (/home/www/nodebb/node_modules/async/lib/async.js:557:34)
at processImmediate [as _immediateCallback] (timers.js:345:15)
info: [app] Shutdown (SIGTERM/SIGINT) Initialised.
info: [app] Database connection closed.
info: [app] Web server closed to connections.
info: [app] Shutdown complete.
[cluster] Child Process (14399) has exited (code: 1)
[cluster] Spinning up another process...
[cluster] Child Process (7088) listening for connections.
[cluster] Child Process (7088) listening for connections.

The post has been successfully published to nodebb though.

For the modification I made in the plugin it's simply in "node_modules/nodebb-plugin-blog-comments/public/templates/comments/admin.tpl" line 29.

a_5mith

My youtube plugin no longer works in 0.5.2 either, seems something has been changed. No idea what mind. Pretty big gotcha. I'm almost being wedgied.

Alex

Ah you too? I thought I was getting crazy, imgbed neither works for me. Well any plugin which parse content doesn't work for me excepted Markdown Parser

a_5mith

@Alex said:

Ah you too? I thought I was getting crazy, imgbed neither works for me. Well any plugin which parse content doesn't work for me excepted Markdown Parser

Yeah, imgbed broke too.

julian

@Alex said:

Ah you too? I thought I was getting crazy, imgbed neither works for me. Well any plugin which parse content doesn't work for me excepted Markdown Parser

That's really worrisome... let's see.

Edit: @a_5mith @Alex How recent is this breakage?

Edit 2: Damn, it's Markdown.

a_5mith

Upstream or regression?

julian

@a_5mith Regression, fixing shortly

? Offline

@julian, fixed it yet?

julian

@Ted no, I had thanksgiving dinner and watched David Tennant's last episode of Doctor Who instead XD

? Offline

@julian but that's the worst episode ever...
Loved Matt Smith in the role, but he's not Tennant

julian

Kind of difficult to fix, because of the change made to Markdown... if the new "noFollow" option is turned on (which it is, by default), a link as returned by markdown would look like this:

<a rel="nofollow" href="//mysite.com">Site</a>

instead of...

<a href="//mysite.com">Site</a>

... which means all of the existing regex won't match any longer. Drat.

Both the "workaround" and the "solution" involve one crappy step: All of the affected plugins need to rewrite their regex or refactor their entire plugin.

The workaround:

All plugins need to rewrite their regex to accommodate the rel="nofollow" addition
If more changes get added to Markdown, all of the affected plugins will break again
= Bad solution

The solution:

See gh#2263

... and to round it out

The cop-out

@julian just removes the rel="nofollow" from Markdown XD ... but that's a step back.

Example 1:

[ { type: 'blockquote_start' },
  { type: 'paragraph',
    text: 'i am using marked.' },
  { type: 'blockquote_end' },
  links: {} ]

Alex

It seems to be a bit laborious to fix it

a_5mith

@julian said:

Kind of difficult to fix, because of the change made to Markdown...

Ill go through my list of plugins and update as necessary.

a_5mith

EDIT: Right, for anyone that's facing this issue, rel="nofollow" needs to be placed BEFORE the href tag. Despite the source code displaying it afterwards...