Skip to content

Add in-app UI to set site cookies (Fix #2182)#2185

Open
bchkv wants to merge 1 commit into
RipMeApp:mainfrom
bchkv:feature/cookie-paste-ui
Open

Add in-app UI to set site cookies (Fix #2182)#2185
bchkv wants to merge 1 commit into
RipMeApp:mainfrom
bchkv:feature/cookie-paste-ui

Conversation

@bchkv

@bchkv bchkv commented Jun 9, 2026

Copy link
Copy Markdown

Category

  • a bug fix (Fix #...)
  • a new Ripper
  • a refactoring
  • a style change/fix
  • a new feature

Description

Adds Configuration → Site cookies... so users can paste a browser Network tab Cookie header without editing rip.properties manually.

  • Cookies are stored in per-domain sidecar files under the config directory (cookies.<domain>), so saving other settings from the UI does not wipe them.
  • Still reads legacy cookies.<domain> entries from rip.properties for backward compatibility.
  • In-dialog step-by-step instructions (Chrome, Edge, Firefox, Safari).
  • 403/401 error messages point users to the new UI.

Related issues:

Testing

Required verification:

  • I've verified that there are no regressions in gradlew test (there are no new failures or errors).
  • I've verified that this change works as intended.
    • Downloads all relevant content.
    • Downloads content from multiple pages (as necessary or appropriate).
    • Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.
  • I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

  • I've added a unit test to cover my change.

Notes: SiteCookieStorageTest passes. Full ./gradlew test has live-site rippper failures that also occur on clean main (9 failures on main vs 6 on this branch); none are introduced by the cookie changes.

@bchkv bchkv marked this pull request as ready for review June 9, 2026 16:24
Stores cookies in config-dir sidecar files with Network-tab paste
instructions. Fixes JWT/JSON cookie parsing and wires Http to load
per-domain cookies.
@bchkv bchkv force-pushed the feature/cookie-paste-ui branch from 87f7f58 to f2e2b35 Compare June 9, 2026 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add in-app instructions for how to set cookies for a domain

1 participant