Deploy for dereckson a playlist generator script on devserver
ClosedPublic

Authored by dereckson on Sat, Jul 21, 15:21.

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
dereckson created this revision.Sat, Jul 21, 15:21
dereckson requested review of this revision.Sat, Jul 21, 15:27
dereckson planned changes to this revision.Sat, Jul 21, 15:50
dereckson added inline comments.
roles/devserver/userland-home/files/dereckson/bin/http2pls
45

If the URL isn't valid, the urlopen function will raise the following exception:

ValueError: unknown url type: 'fdfdf'

File "/usr/local/lib/python3.6/urllib/request.py", line 384, in _parse
  raise ValueError("unknown url type: %r" % self.full_url)

So a try block could handle URL errors here.

50

two \n

dereckson updated this revision to Diff 4225.Sun, Jul 22, 12:47

PEP-8 compliant

dereckson updated this revision to Diff 4226.Sun, Jul 22, 12:58

Handle URL errors

dereckson accepted this revision.Sun, Jul 22, 13:03
dereckson marked an inline comment as done.
This revision is now accepted and ready to land.Sun, Jul 22, 13:03
This revision was automatically updated to reflect the committed changes.