Skip to content

Conversation

nemchik
Copy link
Member

@nemchik nemchik commented Oct 18, 2021

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for opening this pull request! Be sure to follow the pull request template!

@LinuxServer-CI
Copy link
Contributor

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@nemchik nemchik added work-in-progress Stale exempt and removed no-pr-activity labels Apr 13, 2022
@nemchik nemchik force-pushed the standard-base branch 4 times, most recently from 34b6046 to be76b3a Compare August 16, 2022 00:27
@LinuxServer-CI
Copy link
Contributor

@LinuxServer-CI
Copy link
Contributor

1 similar comment
@LinuxServer-CI
Copy link
Contributor

@LinuxServer-CI
Copy link
Contributor

@nemchik nemchik marked this pull request as ready for review August 21, 2022 00:25
@nemchik nemchik requested a review from a team August 21, 2022 00:25
@LinuxServer-CI
Copy link
Contributor

@LinuxServer-CI
Copy link
Contributor

@LinuxServer-CI
Copy link
Contributor

@LinuxServer-CI
Copy link
Contributor

Copy link
Member

@drizuid drizuid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested both a clean install and an upgrade from :latest. when upgrading from :latest the web ui will break until the out of date files are updated. I did not check which of the two files broke it, but I am assuming default.conf is the culprit. It gives a 403 after initial startup following the upgrade. Deleting the 2 files (default.conf and nginx.conf) and restarting resolves the issues. I tested opds using fbreader on android and was able to download and open books fine.

@LinuxServer-CI
Copy link
Contributor

Copy link
Member

@drizuid drizuid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the latest pr resolves the upgrade issues i mentioned before.

@drizuid drizuid merged commit 09ef7f2 into master Dec 26, 2022
@drizuid drizuid deleted the standard-base branch December 26, 2022 15:21
@nemchik nemchik mentioned this pull request Dec 26, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
work-in-progress Stale exempt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants