538309 - ShareThis settings not sticking for hacks.mozilla.org

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: WebOps: Other, task)

Description

[Jay Patel [:jay]]

I am not able to configure the sharethis plugin on Hacks. 

Steps to repro:

1. Admin Console->Settings->ShareThis Options
2. Paste in 
<script type="text/javascript" src="http://w.sharethis.com/button/sharethis.js#publisher=bb56e21e-b1c8-489a-a72f-2bf6c4acf7ed&amp;type=wordpress&amp;style=rotate"></script>
3. Select "No" for both options
4. Click "Update ShareThis Options"
5. Get encryption warning:
Although this page is encrypted, the information you have entered is to be sent over an unencrypted connection and could easily be read by a third party.

Are you sure you want to continue sending this information?

------

I'm guessing that the application can't sent the http request through since we're using SSL for the site. But what do I know?

Can someone on IT help get this resolved.  The sharing functionality is extremely important for the site and I would like to get ShareThis working.

Thanks!

Comment 1

[Shyam Mani [:fox2mike]]

Fixed.

I'm no wordpress/php guru, so my hack to get this working is fairly stupid and I'd like Jeremy to okay this (for the future) or suggest a better way.

So basically, the line that was causing the issue was :

<form id="ak_sharethis" name="ak_sharethis" action="'.get_bloginfo('wpurl').'/wp-admin/index.php" method="post">

The get_bloginfo function when called with wpurl basically pulls out the Wordpress address URL setting from Administration -> Settings -> General.

That's set to http://hacks.mozilla.org since we allow http traffic to the site. I'm guessing the post was failing because anything to do with wp-admin gets pushed to https and the form was trying to post to http.

I replaced the '.get_bloginfo('wpurl').' with https://hacks.mozilla.org and pushed out the plugin code and that made sure it worked.

Jeremy, if you think we can get a better fix, would be nice. I'm thinking the redirects to https are WP internal, because I don't see us doing anything in the Apache config for wp-admin and if so, this should be a bug? 

I was initially thinking it's because we split out wpcontent, but that doesn't seem to be the case at all.

Comment 2

[Jeremy Orem [:oremj]]

I think you want site_url('wp-admin/index.php'). This should probably be fixed upstream.

Comment 3

[Jeremy Orem [:oremj]]

I brought up the issue @ http://forums.sharethis.com/topic.php?id=2386 I'm Not sure how well people watch it though.