Skip to content

feat: increase temporary document size limit to 1MB#4

Open
OhMyMndy wants to merge 1 commit into
tt-rss:mainfrom
OhMyMndy:increase-content-size
Open

feat: increase temporary document size limit to 1MB#4
OhMyMndy wants to merge 1 commit into
tt-rss:mainfrom
OhMyMndy:increase-content-size

Conversation

@OhMyMndy

Copy link
Copy Markdown

I had to increase the limit because some pages were really large.

@supahgreg

Copy link
Copy Markdown
Collaborator

Thanks!

I'm thinking this might be a good opportunity to introduce a config option-- keeping the default at 512000 but allowing an override (e.g. environment variable TTRSS_AF_READABILITY_MAX_DOC_SIZE=1024000).

AUTH_OIDC_VALIDATE_INTERVAL in https://github.com/tt-rss/tt-rss-plugin-auth-oidc/blob/main/init.php could be used as an example, if you're interested in making the change.

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.

2 participants