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.

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

DietPi-Software | Aria2: Config tweak/fix #2538

Merged
merged 2 commits into from
Feb 11, 2019
Merged

Conversation

msongz
Copy link
Contributor

@msongz msongz commented Feb 11, 2019

to fix using 3rd party plugin;
and comment unknown option;

unknown option (systemctl status aria2.service):

   Loaded: loaded (/etc/systemd/system/aria2.service; disabled; vendor preset: enabled)
   Active: active (running) since Tue 2019-02-12 02:12:35 HKT; 5s ago
 Main PID: 3572 (aria2c)
   CGroup: /system.slice/aria2.service
           └─3572 /usr/bin/aria2c --conf-path=/var/lib/dietpi/dietpi-software/installed/aria2.conf

Feb 12 02:12:35 zerow systemd[1]: Started Aria2 (DietPi).
Feb 12 02:12:35 zerow aria2c[3572]: 02/12 02:12:35 [WARN] Unknown option: host=localhost
Feb 12 02:12:35 zerow aria2c[3572]: 02/12 02:12:35 [WARN] Unknown option: cleanup-policy="clean_got"
Feb 12 02:12:35 zerow aria2c[3572]: 02/12 02:12:35 [WARN] Unknown option: cleanup-percent=90%
Feb 12 02:12:35 zerow aria2c[3572]: 02/12 02:12:35 [WARN] Unknown option: metalink-servers=7
Feb 12 02:12:35 zerow aria2c[3572]: 02/12 02:12:35 [WARN] Unknown option: [limit-options]
Feb 12 02:12:35 zerow aria2c[3572]: 02/12 02:12:35 [WARN] Unknown option: [unlimit-options]
Feb 12 02:12:35 zerow aria2c[3572]: 02/12 02:12:35 [NOTICE] IPv4 RPC: listening on TCP port 6800
Feb 12 02:12:35 zerow aria2c[3572]: 02/12 02:12:35 [NOTICE] IPv6 RPC: listening on TCP port 6800

offical ref:
https://aria2.github.io/manual/en/html/aria2c.html#cmdoption-rpc-allow-origin-all

3rd plugin:
https://github.com/acgotaku/BaiduExporter

to fix using 3rd party plugin;
and comment unknown option;
@MichaIng
Copy link
Owner

MichaIng commented Feb 11, 2019

@msongz
Many thanks for your PR.

On which device/distro version did you test?

On Stretch (and above) we install from APT repo, on Jessie we use an own compiled binary. Perhaps the unknown options (and sectioning in config file) does only work on one of them. Also perhaps the settings names have changed, so instead of removing we could rename them, in case dependant on Jessie vs Stretch.

  • But yeah easy to test. Will check on VMs.

Verified deprecated settings on Jessie, Stretch and Buster, all the same. Those settings seem to be relics from old times 😉:

root@VM-Jessie:~# systemctl status aria2
● aria2.service - Aria2 (DietPi)
   Loaded: loaded (/etc/systemd/system/aria2.service; disabled)
   Active: active (running) since Mon 2019-02-11 21:03:42 CET; 26s ago
 Main PID: 6805 (aria2c)
   CGroup: /system.slice/aria2.service
           └─6805 /usr/local/bin/aria2c --conf-path=/var/lib/dietpi/dietpi-software/installed/aria2.conf

Feb 11 21:03:42 VM-Jessie systemd[1]: Started Aria2 (DietPi).
Feb 11 21:03:42 VM-Jessie aria2c[6805]: 02/11 21:03:42 [WARN] Unknown option: host=localhost
Feb 11 21:03:42 VM-Jessie aria2c[6805]: 02/11 21:03:42 [WARN] Unknown option: cleanup-policy=clean_got
Feb 11 21:03:42 VM-Jessie aria2c[6805]: 02/11 21:03:42 [WARN] Unknown option: cleanup-percent=90%
Feb 11 21:03:42 VM-Jessie aria2c[6805]: 02/11 21:03:42 [WARN] Unknown option: metalink-servers=40
Feb 11 21:03:42 VM-Jessie aria2c[6805]: 02/11 21:03:42 [WARN] Unknown option: [limit-options]
Feb 11 21:03:42 VM-Jessie aria2c[6805]: 02/11 21:03:42 [WARN] Unknown option: [unlimit-options]
Feb 11 21:03:42 VM-Jessie aria2c[6805]: 02/11 21:03:42 [NOTICE] IPv4 RPC: listening on TCP port 6800
Feb 11 21:03:42 VM-Jessie aria2c[6805]: 02/11 21:03:42 [ERROR] IPv6 RPC: failed to bind TCP port 6800
Feb 11 21:03:42 VM-Jessie aria2c[6805]: Exception: [SocketCore.cc:312] errorCode=1 Failed to bind a socket, cause: Name or service not known

@MichaIng MichaIng changed the title add aria2 config option: rpc allow origin all DietPi-Software | Aria2: Config tweak/fix Feb 11, 2019
@MichaIng
Copy link
Owner

MichaIng commented Feb 11, 2019

Ahhh, found where we have those settings from: https://github.com/hzamani/aria2web/blob/master/config/aria2.conf

Is it possible that the Aria2 web UI uses these settings, while the Aria2 binary doesn't know them? Actually no clean solution, instead the web UI should use an own config file... However if those do not hurt, but show those warnings, it is acceptable?

dietpi/dietpi-software Outdated Show resolved Hide resolved
+ DietPi-Software | Aria2: Remove deprecated settings (instead of comment only); Remove doubled settings; Add bandwidth limit info
@MichaIng
Copy link
Owner

MichaIng commented Feb 11, 2019

Passed local testing, will merge.
Changelog: Fourdee@03394b9

@MichaIng MichaIng merged commit 48c8d83 into MichaIng:dev Feb 11, 2019
@msongz
Copy link
Contributor Author

msongz commented Feb 12, 2019

wow! very detail development! thanks!

@MichaIng MichaIng mentioned this pull request Mar 3, 2019
@MichaIng MichaIng mentioned this pull request Mar 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants