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

remove lj.rossia.org as xpost destination (while keeping it in the user tag) #2095

Closed
rahaeli opened this issue Apr 12, 2017 · 8 comments
Closed

Comments

@rahaeli
Copy link
Contributor

rahaeli commented Apr 12, 2017

https://www.dreamwidth.org/support/see_request?id=36290

From the error message, I'm going to guess that the problem is either a) lj.rossia.org doesn't have a working version of the protocol running, or b) they do but the xposter is having trouble handling it because it's a three-level domain and not a two-level domain. (I know that was a problem with adding LJR to the user tag.)

Obviously we can't fix A, but if it's B, that's something we can do something about!

@kareila
Copy link
Member

kareila commented Apr 12, 2017

It's definitely their protocol that's broken. I tested it with jbackup.pl and got the same error.

do_sync: calling syncitems with lastsync = none yet
		xmlrpc_call_helper: LJ.XMLRPC.getchallenge
xmlrpc_call_helper error:
	String: Failed to access class (LJ::XMLRPC): Can't locate LJ/XMLRPC.pm in @INC (@INC contains: /home/lj-admin/lj/cgi-bin [etc. etc.]) at (eval 1326) line 2.

	Code: Client
Aborted.

@kareila kareila closed this as completed Apr 12, 2017
@rahaeli rahaeli changed the title can't add lj.rossia.org as xpost destination remove lj.rossia.org as xpost destination (while keeping it in the user tag) Apr 12, 2017
@rahaeli
Copy link
Contributor Author

rahaeli commented Apr 12, 2017

Reopened issue and changed subject accordingly! Thanks for the investigation.

@rahaeli
Copy link
Contributor Author

rahaeli commented Apr 12, 2017

Although we should probably wait a few days and see if they fix it.

@kareila
Copy link
Member

kareila commented Apr 12, 2017

Ah, I didn't realize it was in the list of defaults. I assumed they were just using the "Other Site" option.

To clarify: on /manage/externalaccount we need to remove the "LJRossia" option from the site menu.

But we probably don't want to delete it from the backend, or else existing crosspost setups will be very confused.

@rahaeli
Copy link
Contributor Author

rahaeli commented Apr 12, 2017 via email

@kaberett
Copy link
Contributor

Didn't that literally just get done for the insanejournal meltdown?

@kareila
Copy link
Member

kareila commented Apr 12, 2017

I think we only temporarily disabled imports for that case, not crossposts.

kareila added a commit to kareila/dreamwidth that referenced this issue Apr 13, 2017
This removes LJRossia from the "Add External Account" form
and preemptively fails any attempts to crosspost there.

The reason for doing this is because the crosspost attempt
is returning a confusing message to the user about not
being able to locate LJ/XMLRPC.pm on the remote site.
kareila added a commit to kareila/dreamwidth that referenced this issue Apr 13, 2017
It turns out that failing the job doesn't automatically
notify the user, so make sure to send a notification
to inform the user of the current situation.
zorkian added a commit that referenced this issue Apr 15, 2017
[#2095] temporarily disable LJRossia crossposts
@kaberett
Copy link
Contributor

(Noting that the dw_maint post does say crossposting was temporarily disabled, but ???? https://dw-maintenance.dreamwidth.org/75223.html)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

4 participants