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.

Sign up for GitHub

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

Jump to bottom

Feat: LazyLibrarian #772

Open
wants to merge 63 commits into
base: develop
Choose a base branch
from
Open

Feat: LazyLibrarian #772

wants to merge 63 commits into from

Conversation

Aethaeran
Copy link
Contributor

Description

Add requested feature lazy librarian

Fixes issues:

  • Issue #nr...
  • Un-tracked issue...

Proposed Changes:

  • Changed xyz...

Change Categories

  • Bug fix
  • New feature
  • Breaking change
  • Change in functions
  • Project structure
  • Contribution flow

Checklist

  • Branch was made off the develop branch and the PR is targetting the develop branch
  • Docs have been made OR are not necessary
    • PR link:
  • Changes to panel have been made OR are not necessary
    • PR link:
  • Code is formatted (See more)
  • Code conforms to project structure (See more)
  • Shellcheck isn't screaming (See more)
  • Prints to terminal are handled (See more)
  • I have commented my code, particularly in hard-to-understand areas
  • Testing was done
    • Tests created or no new tests necessary
    • Tests executed

Test scenarios

Architectures

amd64 armhf arm64 Unspecified
Focal
Bionic
Buster
Stretch
Raspbian ⚫️ ⚫️ ⚫️

✅❎ Passed

🛠🛠 TODO

❌❌ Currently failing

@Aethaeran
Update .gitignore
94df860
@Aethaeran
Create lazylibrarian.sh
965e0f7
@Aethaeran
added reference to jdownloader script
08a33a7
@Aethaeran
add box list description
d31a18e
@Aethaeran
Create remove/lazylibrarian.sh
512f655
@Aethaeran
Create nginx/lazylibrarian.sh
dbde969
@Aethaeran
Update lazylibrarian.sh
25d5a03
@Aethaeran
Update lazylibrarian.sh
4f97338
@Aethaeran
Update remove/jdownloader.sh
9d8e226
@Aethaeran
Update install/lazylibrarian.sh
17b2434
@Aethaeran
Update lazylibrarian.sh
a7ecd1e
@Aethaeran
Update lazylibrarian.sh
92ac221
@Aethaeran
not gonna configure multi-users after all. Toss that
ad38aa2
@Aethaeran
add todo about venv. Looks like it has some dependencies after all
ba1728b
@Aethaeran
Update nginx
700547b
@Aethaeran
Update remove
67ac363
@Aethaeran
Update install
5c210bf
@Aethaeran
Don't expand variables plox
b9d45c1
@Aethaeran
add "Web Root" aka "Base URL" setting
5d28586
@Aethaeran
Create test
696b66c
@Aethaeran
Update remove
46a7437
@Aethaeran
Update install
9ea9d18
@Aethaeran
some shellcheck diables, some TODOs, some TODOnes.
9933aae
@Aethaeran
add git auto_update to initial config
bea9534
@Aethaeran
added simple logic to remove/lazylibrarian
a56fee6
@Aethaeran
simple logic to verify nginx conf needs creating in nginx/lazylibrari…
fe3991f 
…an.sh
@Aethaeran
add logic to exit if git clone fails
d8bc91e
@Aethaeran
save database and config to master user's .config for easier backup\r…
8c8c844 
…estores
@Aethaeran
now utilizes venv
3749dbb
@Aethaeran
remove making vev_dir ahead of time. Unnecessary
93447ae
flying-sausages and others added 30 commits August 22, 2021 13:03
@flying-sausages
feat: environment variables, formatting
18d5fe0
@flying-sausages
fix: double-slash
b979a12
@flying-sausages
fix: echos
de05d97
@flying-sausages
fix: one-line ExecStart so we can run sed on it later
c24d268
@flying-sausages
style: clean up comments
2784b8a
@flying-sausages
fix: perms
2401dde
@flying-sausages
fix: shfmt
a79210e
@flying-sausages
fix: entire remover
6015bac
@flying-sausages
fix: nginx script
8b5d2b1
@flying-sausages
fix: test
374ee9a
@Aethaeran
Revert "fix: test"
847ac42 
This reverts commit 374ee9a.
@Aethaeran
Revert "fix: nginx script"
ec31166 
This reverts commit 8b5d2b1.
@Aethaeran
Revert "fix: entire remover"
e84d184 
This reverts commit 6015bac.
@Aethaeran
Revert "style: clean up comments"
68194cb 
This reverts commit 2784b8a.
@Aethaeran
Revert "fix: one-line ExecStart so we can run sed on it later"
e615feb 
This reverts commit c24d268.
@Aethaeran
added journalctl test
f2888fe
@Aethaeran
Update test/lazylibrarian.sh
7a2a291
@Aethaeran
Update test
972e44b
@Aethaeran
Fix missing pip, and add download_dir
62df1c9 
Was getting these errors in Deb 10. Adding `apt_install "python3-venv"` rectifies this.

```
    # 23-Aug-2021 14:00:34 - WARNING :: MAIN : __init__.py:initialize:976 : urllib3: missing
    # 23-Aug-2021 14:00:34 - WARNING :: MAIN : __init__.py:initialize:976 : apprise: library missing
    # 23-Aug-2021 14:00:34 - WARNING :: MAIN : __init__.py:initialize:976 : python imaging: library missing
    # 23-Aug-2021 14:00:34 - WARNING :: MAIN : __init__.py:initialize:976 : pyOpenSSL: module missing
```
Also added a default download dir because LazyLib will throw an error in it's log otherwise.
@Aethaeran
general restructuring
e1d9283
@Aethaeran
Moved _get_app_owner logic to sources/functions/users
f854bd8
@Aethaeran
Fix useless cat
8e8b385
@Aethaeran
fix typo
d039401
@Aethaeran
Update users
0013c1c
@Aethaeran
Added PIDFile to service
e1fa9fa
@Aethaeran
Maybe that cat wasn't useless after all
54cbadd
@Aethaeran
couple comments
8d5f874
@Aethaeran
Removed TODOones. shortend references.
594a4a8
@Aethaeran
Merge branch 'develop' into feat/lazy-librarian
8f71be7
@Aethaeran
add TODOs
3f4ea11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Aethaeran @flying-sausages