Import from SongSelect no longer works (website updated)

Bug #1608194 reported by Jayvin
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenLP
Status tracked in Trunk
2.4
Fix Released
Medium
Raoul Snyman
Trunk
Fix Committed
Medium
Raoul Snyman

Bug Description

The SongSelect website has recently been updated, and since then, all login attempts from OpenLP say Login details incorrect, though using the same credentials on the SongSelect website work correctly.

Windows 10 Home
OpenLP version 2.4

- File > Import > CCLI SongSelect
- Enter valid credentials
- "Invalid credentials"

Related branches

Revision history for this message
Tim Schulz (timschulz91) wrote :

Same for Ubuntu 16.04 and Windows 8.1.

Changed in openlp:
importance: Undecided → Medium
status: New → Confirmed
Changed in openlp:
assignee: nobody → Raoul Snyman (raoul-snyman)
Revision history for this message
maarts (maarts) wrote :

Under Ubuntu with openlp 2.4.3 runs the ccli import search in endless loop. Try it with "ocean". I killed the search after 650 results...

Revision history for this message
Tim Schulz (timschulz91) wrote :

I know this is an old bug and it was fixed in between, but it's back.

Trying to import songs from CCLI, I get
"There was a problem logging in, perhaps your username or password is incorrect?"

Happens on both Windows 10 and Ubuntu 16.04, both running 2.4.5.

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

Yes, we know.

Revision history for this message
Tim Schulz (timschulz91) wrote :

Fix intermittently but now breaks on search (after login).

I don't know how to submit a PR on launchpad, but adding a `or is None` on line 130 of `html.__init__.py` in `unescape()` fixed it. It now reads

```python
if '&' not in s or s is None:
    return s
return _charref.sub(_replace_charref, s)
```

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

That's Python's code, not ours, but it probably means we need to filter the string before it goes to unescape().

Revision history for this message
Tomas Groth (tomasgroth) wrote :

Fix committed

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.