This repository was archived by the owner on Feb 22, 2021. It is now read-only.
Authentication Cookie#296
Closed
Etuldan wants to merge 2 commits into
Closed
Conversation
added 2 commits
May 1, 2015 11:32
Add authentication cookie for private/protected contents. Users have to enter (if they want) the cookie Name and Value in the Feed option. The cookie Name and Value should be grabbed manually from another browser where the user has previously logon on the website.
Owner
|
I'm sorry but as I have declared into the project description, unfortunately I don't have the time anymore to maintain this software and thus won't include your change. If you're motivated, I encourage you to maintain your own fork! |
Contributor
Author
|
About the fork (thay I'll probably do), should I use the same Application Name ? |
Owner
|
To avoid any confusion on the Play Store I would use another name for your fork. Don't you think it's better this way? |
Etuldan
pushed a commit
to Etuldan/spaRSS
that referenced
this pull request
May 7, 2015
Project rename from Flym to spaRSS. PackageID change from net.fred.feedex to net.etuldan.sparss Logo (PNG and SVG) text modified see FredJul#296 to understand why I started a fork (with a new name)
Contributor
Author
|
As you can see, I found a new name :) |
Contributor
Author
|
The fork is available for free on the PlayStore (you will find the link on my GitHub project page) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
For some website (like MediaPart.fr), we need to be log with a paid account to view the full content.
Indeed, the website check for a cookie.
Flym was not able to grab the full content of the news (when "Retrieve the full text" is set), as there was no cookie set.
I add in this fork the ability to manually set a cookie (name/value) for a specified feedEntry.
The user have to grab the cookie information from another browser (on the Android device or not) after he logs on the WebSite. Then he should copy/paste this informations to the FeedEntry configuration on Flym.
Also, I didn't translate the news Strings that I add ( Authentication Cookie, Name, Value).