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

Github repositories search broken #1430

Closed
virtadpt opened this issue Jan 21, 2020 · 3 comments · Fixed by #1431
Closed

Github repositories search broken #1430

virtadpt opened this issue Jan 21, 2020 · 3 comments · Fixed by #1431
Labels
Bug-Report Confirmed bug report

Comments

@virtadpt
Copy link

Describe the bug
When using the Github Repositories search bridge, a broken RSS feed with largely unpopulated entries is returned. This occurs for Plaintext, MRSS, JSON, HRML, and ATOM. This is occurring on RSS Bridge version git.master.46b9879.

To Reproduce
Steps to reproduce the behavior:

  1. RSS-Bridge
  2. Scroll down to Github Repositories Search. Enter a search query (e.g., "matrix client").
  3. Click on one of the feed buttons (Plaintext, MRSS, JSON, HRML, or ATOM).
  4. Entries will look like the following (here, ATOM):
	<entry>
		<title type="html">No description</title>
		<published>2020-01-21T21:23:28+00:00</published>
		<updated>2020-01-21T21:23:28+00:00</updated>
		<id>urn:sha1:3c9275167e62a81a9aba04ed64e5bfe153fa2500</id>
		
		
		<content type="html">No description&lt;br /&gt;</content>
		
		
	</entry>

	<entry>
		<title type="html">No descriptionTags : matrix qt5 chat client c-plus-plus qt sdk </title>
		<published>2020-01-21T19:03:27+00:00</published>
		<updated>2020-01-21T19:03:27+00:00</updated>
		<id>urn:sha1:71927724f54e855283b3fc03aa86417fff0aad0d</id>
		
		
		<content type="html">No description&lt;br /&gt;Tags : &lt;a href="https://github.com/topics/matrix"&gt;matrix&lt;/a&gt; &lt;a href="https://github.com/topics/qt5"&gt;qt5&lt;/a&gt; &lt;a href="https://github.com/topics/chat"&gt;chat&lt;/a&gt; &lt;a href="https://github.com/topics/client"&gt;client&lt;/a&gt; &lt;a href="https://github.com/topics/c-plus-plus"&gt;c-plus-plus&lt;/a&gt; &lt;a href="https://github.com/topics/qt"&gt;qt&lt;/a&gt; &lt;a href="https://github.com/topics/sdk"&gt;sdk&lt;/a&gt; </content>
		
		<category term="   matrix "/>
<category term="   qt5 "/>
<category term="   chat "/>
<category term="   client "/>
<category term="   c-plus-plus "/>
<category term="   qt "/>
<category term="   sdk "/>

	</entry>

Expected behavior
ATOM feed entries with valid, human readable titles and descriptions.

Screenshots
Can be furnished upon request.

Desktop (please complete the following information):
Work:

  • OS: Mac OS X
  • Browser: Chrome
  • Version 79.0.3945.88 (Official Build) (64-bit)

Home:

  • OS: Arch Linux
  • Browser: Firefox
  • Version 72.0.1 (64-bit)

Server:

@virtadpt virtadpt added the Bug-Report Confirmed bug report label Jan 21, 2020
AntoineTurmel pushed a commit to AntoineTurmel/rss-bridge that referenced this issue Jan 21, 2020
@AntoineTurmel
Copy link
Contributor

I made a patch, basically GitHub changed some html tags and css class.

@virtadpt
Copy link
Author

Thank you - I'll do a git pull and test it out tonight.

@virtadpt
Copy link
Author

When it's merged, let me know. It didn't come down last night.

teromene pushed a commit that referenced this issue Jan 31, 2020
infominer33 pushed a commit to web-work-tools/rss-bridge that referenced this issue Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug-Report Confirmed bug report
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants