Import from SongSelect no longer works (website updated)

Bug #1608194 reported by Jayvin on 2016-07-31
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
OpenLP
Status tracked in Trunk
2.4
Medium
Raoul Snyman
Trunk
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

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)
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...

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.

Raoul Snyman (raoul-snyman) wrote :

Yes, we know.

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)
```

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().

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

Duplicates of this bug

Other bug subscribers