| Wednesday, June 23rd, 2021, 00:08 UTC | ||
| [00:08:56] | bill6502: | Hmmm: api2test.py --endpoint Myth/GetHostName --oneline |
| [00:08:58] | bill6502: | {'String': 'ofc0'} |
| [00:09:00] | bill6502: | {'HostName': 'ofc0' |
| [00:09:29] | bill6502: | Never liked the old 'String', but it's in the wild now... |
| [00:09:52] | bill6502: | The missing trailing '}' is my cut/paste error |
| [03:38:05] | bill6502 (bill6502!~bill@mythtv/developer/bill6502) has left #mythtv () | |
| [04:37:15] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has joined #mythtv | |
| [07:23:32] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has quit (Quit: Konversation terminated!) | |
| [07:41:43] | markspieth2 (markspieth2!~markspiet@202.169.118.74) has joined #mythtv | |
| [07:41:43] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has joined #mythtv | |
| [07:41:43] | markspieth2 (markspieth2!~markspiet@202.169.118.74) has quit (Changing host) | |
| [07:41:43] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v markspieth2 | |
| [07:50:42] | DevMythNotifyBot: | COMMIT: stuarta pushed 3 commit(s) to branch devel/http: https://github.com/MythTV/mythtv/compare/4fe4 . . . 5bd6c6a3d9be |
| [07:54:25] | stuarta: | i wonder why 2 of the eit fixup tests are failing only on archlinux |
| [07:57:27] | stuarta: | i don't have an arch linux box handy to test on, and i'm not spending another 2 days of my life building one |
| [08:04:35] | stuarta: | ahh... cloud images, this might work |
| [09:27:13] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has joined #mythtv | |
| [09:27:14] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v markspieth2 | |
| [09:45:53] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has joined #mythtv | |
| [09:46:58] | stuarta: | well i can reproduce the issue |
| [09:56:42] | xris (xris!~xris@mythtv/developer/xris) has quit (Quit: Textual IRC Client: www.textualapp.com) | |
| [10:05:09] | stuarta: | just not a clue why |
| [10:05:47] | stuarta: | something in the series regexp isn't capturing the series number properly |
| [10:05:50] | stuarta: | just not a clue why |
| [10:07:08] | stuarta: | S01 Ep39 -> finding season 0, ep 29 |
| [10:07:28] | stuarta: | S4 Ep3 -> finding season 0, ep 3 |
| [11:35:42] | ** hampton is glad he added all those EIT tests ** | |
| [11:36:17] | hampton: | Last time my builders ran (couple of weeks ago) the arch builder completed successfully. |
| [11:37:11] | hampton: | I'll try and find time later today to update that system and force a rebuild, and see what happens. |
| [11:46:51] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has quit (Ping timeout: 256 seconds) | |
| [12:11:37] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has joined #mythtv | |
| [12:11:37] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v markspieth2 | |
| [12:15:51] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has quit (Ping timeout: 246 seconds) | |
| [12:30:30] | markspieth2 (markspieth2!~markspiet@202.169.118.74) has joined #mythtv | |
| [12:30:30] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has joined #mythtv | |
| [12:30:30] | markspieth2 (markspieth2!~markspiet@202.169.118.74) has quit (Changing host) | |
| [12:30:30] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v markspieth2 | |
| [13:12:27] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [13:12:27] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [13:17:33] | peterbennett: | bill6502: Your result of now getting HostName instead of String is consistent with the Video/GetSavedBookMark which previously returned "long" now returns "SavedBookmark" as the tag. |
| [13:18:02] | ** stuarta likes consistency ** | |
| [13:40:55] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [14:26:29] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [14:26:29] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [14:35:40] | stuarta: | the new webserver is start up even after the scheduler. it may need to start earlier in future |
| [14:37:07] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [14:38:02] | markspieth2 (markspieth2!~markspiet@202.169.118.74) has joined #mythtv | |
| [14:38:02] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has joined #mythtv | |
| [14:38:02] | markspieth2 (markspieth2!~markspiet@202.169.118.74) has quit (Changing host) | |
| [14:38:02] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v markspieth2 | |
| [14:42:30] | stuarta: | odd the http method code is doing |= so adding methods=POST *should* add POST to the allowed HTTP verbs |
| [14:46:23] | stuarta: | and it does :) |
| [14:52:20] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [14:52:21] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [14:54:42] | peterbennett: | stuarta: I did not add POST to the methods. Will I be able to test SOAP input? Is there a change that will allow it or should I add the POST,GET,HEAD ? |
| [14:54:55] | peterbennett: | What is the eventual plan for that? |
| [15:00:35] | warpme_ (warpme_!uid391875@id-391875.brockwell.irccloud.com) has joined #mythtv | |
| [15:36:19] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [15:36:53] | bill6502 (bill6502!~bill@mythtv/developer/bill6502) has joined #mythtv | |
| [15:36:53] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v bill6502 | |
| [15:37:45] | bill6502: | Good morning-5 |
| [15:38:37] | bill6502: | I've got a tools that runs json on the old port and new, then xml on both |
| [15:38:58] | bill6502: | here's the output for my 1st services: https://pastebin.com/raw/Bh7MTqyX |
| [15:39:57] | bill6502: | the <--------------------------- mark the differences. I agree that they're better (e.g. Host rather than String), |
| [15:41:24] | bill6502: | stuarta: peterbennett: Do we announce differences to developers can make changes or try to make V2 match V2? |
| [15:41:57] | bill6502: | s/to/so |
| [15:45:51] | stuarta: | bah, this is supposed to OR the new flags in |
| [15:46:09] | stuarta: | HTTPResp: 'GET' is not allowed for 'GetVideo' (Allowed: POST, OPTIONS) <- this is with "methods=POST" |
| [15:47:27] | stuarta: | there's that comment i read -> // N.B. GET and HEAD must be supported for HTTP/1.1 |
| [15:54:05] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [15:54:05] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [15:57:10] | stuarta: | peterbennett: ah you are back. the plan is to have the api classes say methods="POST" if they need POST support. the code is meant to OR in any methods specified in the class, but it's not for some reason. |
| [16:04:07] | peterbennett: | OK so leave it without specifying for now |
| [16:04:16] | stuarta: | no, please specify method=POST |
| [16:04:27] | stuarta: | if you need POST support on a call |
| [16:04:37] | stuarta: | s/call/method |
| [16:05:30] | stuarta: | i'll investigate why it's not actually OR'ing in the methods |
| [16:06:12] | peterbennett: | Sure, but for these methods we do not need POST so I will leave it out for now. |
| [16:06:41] | stuarta: | correct, but if you need SOAP support, then you must accept POST |
| [16:11:08] | peterbennett: | I can wait until you sorted out why its not working. |
| [16:11:56] | stuarta: | i would need to add it back in anyway (i'm currently having to local patch my tree just to test), so if it will eventually need it, please can you add it now? |
| [16:15:59] | peterbennett: | All the services support soap so they will all need it. |
| [16:17:35] | peterbennett: | So we should specify GET,POST,HEAD for those that get data and POST for those that update. |
| [16:18:17] | stuarta: | i'm beginning to think that is the way it's designed |
| [16:18:27] | peterbennett: | Or else change the default from GET,HEAD,OPTIONS to GET,POST,HEAD |
| [16:18:44] | stuarta: | but it's odd that despite the comment saying http 1.1 always must support GET and HEAD it's not true |
| [16:20:55] | stuarta: | the code is automatically enabling POST for any method start with "Set" |
| [16:21:37] | stuarta: | and defaulting to GET HEAD for anything named "Get*" |
| [16:22:38] | stuarta: | it also adds OPTIONS for everything, as that is how you can ask what methods are supported. |
| [16:22:56] | stuarta: | peterbennett: i think the easiest is to add POST support to Get Methods by default |
| [16:23:47] | peterbennett: | OK |
| [16:24:03] | stuarta: | then it's mainly just "do the right thing" |
| [16:24:22] | peterbennett: | So defaulting to GET HEAD for anything named "Get*" |
| [16:24:26] | stuarta: | anything outside the Get/Set naming scheme will explicitly need to set what it supports |
| [16:24:38] | stuarta: | yes, and i'm just adding POST to that list |
| [16:24:42] | peterbennett: | change to and defaulting to GET HEAD POST for anything named "Get*" |
| [16:24:51] | stuarta: | yes |
| [16:26:16] | peterbennett: | A quick glance suggests that all the get methods start with "Get" |
| [16:26:47] | peterbennett: | So that will cover it for 99% or 100% of cases. |
| [16:27:42] | stuarta: | yup |
| [16:32:43] | DevMythNotifyBot: | COMMIT: stuarta pushed 1 commit(s) to branch devel/http: https://github.com/MythTV/mythtv/compare/5bd6 . . . fe34c418716a |
| [16:35:34] | stuarta: | right, i'm done for now. |
| [16:41:11] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [16:56:14] | DevMythNotifyBot: | COMMIT: kmdewaal pushed 1 commit(s) to branch master: https://github.com/MythTV/mythtv/compare/e945 . . . ad6f2ff679e0 |
| [17:13:37] | MythBuild: | Build [#1272](https://code.mythtv.org/buildbot/#builders/24/builds/1272) of `master-fedora-aarch64` 4failed. |
| [17:14:10] | MythBuild: | Build [#1123](https://code.mythtv.org/buildbot/#builders/22/builds/1123) of `master-ubuntu-18_04–64bit` 4failed. |
| [17:15:18] | MythBuild: | Build [#699](https://code.mythtv.org/buildbot/#builders/160/builds/699) of `master-ubuntu-20_04–64bit` 4failed. |
| [17:15:48] | MythBuild: | Build [#1268](https://code.mythtv.org/buildbot/#builders/30/builds/1268) of `master-ubuntu-lts-64bit` 4failed. |
| [17:16:07] | MythBuild: | Build [#17](https://code.mythtv.org/buildbot/#builders/233/builds/17) of `master-ubuntu-21_04–64bit` 4failed. |
| [17:16:52] | MythBuild: | Build [#1295](https://code.mythtv.org/buildbot/#builders/12/builds/1295) of `master-fedora-armv7hl` 4failed. |
| [17:17:29] | MythBuild: | Build [#1043](https://code.mythtv.org/buildbot/#builders/117/builds/1043) of `master-debian-bullseye-64bit` 4failed. |
| [17:18:02] | MythBuild: | Build [#1349](https://code.mythtv.org/buildbot/#builders/45/builds/1349) of `master-freebsd12–64bit` 4failed. |
| [17:19:31] | MythBuild: | Build [#1304](https://code.mythtv.org/buildbot/#builders/11/builds/1304) of `master-fedora-rawhide-64bit` 4failed. |
| [17:19:33] | MythBuild: | Build [#24](https://code.mythtv.org/buildbot/#builders/223/builds/24) of `master-f34–64bit` 4failed. |
| [17:21:09] | MythBuild: | Build [#313](https://code.mythtv.org/buildbot/#builders/197/builds/313) of `master-f33–64bit` 4failed. |
| [17:24:00] | MythBuild: | Build [#18](https://code.mythtv.org/buildbot/#builders/228/builds/18) of `master-freebsd13–64bit` 4failed. |
| [17:29:42] | MythBuild: | Build [#1258](https://code.mythtv.org/buildbot/#builders/76/builds/1258) of `master-osx-64bit` 4failed. |
| [17:42:45] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [17:42:45] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [17:50:06] | MythBuild: | Build [#1053](https://code.mythtv.org/buildbot/#builders/112/builds/1053) of `master-debian-buster-rpi2-armv7l` 4failed. |
| [17:52:19] | MythBuild: | Build [#745](https://code.mythtv.org/buildbot/#builders/156/builds/745) of `master-f32–64bit` 4failed. |
| [18:03:02] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [18:14:07] | MythBuild: | Build [#1280](https://code.mythtv.org/buildbot/#builders/105/builds/1280) of `master-debian-buster-64bit` 4failed. |
| [18:25:03] | sphery (sphery!~mdean@mythtv/developer/sphery) has joined #mythtv | |
| [18:25:03] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v sphery | |
| [18:25:38] | MythBuild: | Build [#320](https://code.mythtv.org/buildbot/#builders/200/builds/320) of `master-ubuntu-20_10–64bit` 4failed. |
| [18:48:43] | stuarta: | oops, test case needs updating |
| [19:06:00] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [19:06:00] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [19:24:50] | MythBuild: | Build [#1061](https://code.mythtv.org/buildbot/#builders/123/builds/1061) of `master-centos8–64bit` 4failed. |
| [19:35:22] | DevMythNotifyBot: | COMMIT: kmdewaal pushed 1 commit(s) to branch master: https://github.com/MythTV/mythtv/compare/ad6f . . . 409e2c953628 |
| [19:36:26] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [19:38:47] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [19:38:48] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [19:39:13] | Steve-Goodey (Steve-Goodey!~steve@2a00:23c5:7d83:6501:6ea1:20d:5872:d872) has quit (Quit: Konversation terminated!) | |
| [19:47:36] | bill6502: | peterbennett: FYI, Robert McNamara made some 'rules' about naming here: https://code.mythtv.org/trac/changeset/ce52b5e so GetSomeEndpoint looks good. ParseISODDate exists in the Myth Service |
| [19:50:21] | MythBuild: | Build [#1124](https://code.mythtv.org/buildbot/#builders/22/builds/1124) of `master-ubuntu-18_04–64bit` 3completed successfully. |
| [19:52:32] | MythBuild: | Build [#1273](https://code.mythtv.org/buildbot/#builders/24/builds/1273) of `master-fedora-aarch64` 3completed successfully. |
| [19:53:03] | peterbennett: | bill6502: I see that. I was not aware of it, I guess it is of most interest to people adding new methods |
| [19:53:43] | glenb77 (glenb77!~glenb77@2603-8000-b440-c812-0000-0000-0000-0331.res6.spectrum.com) has joined #mythtv | |
| [19:53:51] | MythBuild: | Build [#700](https://code.mythtv.org/buildbot/#builders/160/builds/700) of `master-ubuntu-20_04–64bit` 3completed successfully. |
| [19:54:34] | MythBuild: | Build [#1062](https://code.mythtv.org/buildbot/#builders/123/builds/1062) of `master-centos8–64bit` 3completed successfully. |
| [19:54:43] | bill6502: | Found it more by accident myself. Then broke the rules by using the ManageSomeEndpoint rather than multiple ones for changes/additions/deletions |
| [19:55:19] | MythBuild: | Build [#1269](https://code.mythtv.org/buildbot/#builders/30/builds/1269) of `master-ubuntu-lts-64bit` 3completed successfully. |
| [19:56:04] | MythBuild: | Build [#1296](https://code.mythtv.org/buildbot/#builders/12/builds/1296) of `master-fedora-armv7hl` 3completed successfully. |
| [19:56:06] | MythBuild: | Build [#18](https://code.mythtv.org/buildbot/#builders/233/builds/18) of `master-ubuntu-21_04–64bit` 3completed successfully. |
| [19:56:07] | MythBuild: | Build [#1044](https://code.mythtv.org/buildbot/#builders/117/builds/1044) of `master-debian-bullseye-64bit` 3completed successfully. |
| [19:56:11] | peterbennett: | bill6502: It should be in the wiki. In the pages on service development |
| [19:56:31] | bill6502: | https://www.mythtv.org/wiki/Services_API_Development_Guide |
| [19:57:02] | bill6502: | was just looking it up, I recall someone ;) wrote that a bit ago. I'll add to it. |
| [19:57:10] | MythBuild: | Build [#19](https://code.mythtv.org/buildbot/#builders/228/builds/19) of `master-freebsd13–64bit` 3completed successfully. |
| [19:57:47] | MythBuild: | Build [#1350](https://code.mythtv.org/buildbot/#builders/45/builds/1350) of `master-freebsd12–64bit` 3completed successfully. |
| [19:58:37] | MythBuild: | Build [#321](https://code.mythtv.org/buildbot/#builders/200/builds/321) of `master-ubuntu-20_10–64bit` 3completed successfully. |
| [19:58:51] | MythBuild: | Build [#25](https://code.mythtv.org/buildbot/#builders/223/builds/25) of `master-f34–64bit` 3completed successfully. |
| [20:00:20] | MythBuild: | Build [#314](https://code.mythtv.org/buildbot/#builders/197/builds/314) of `master-f33–64bit` 3completed successfully. |
| [20:06:07] | bill6502: | wiki done |
| [20:06:25] | MythBuild: | Build [#1259](https://code.mythtv.org/buildbot/#builders/76/builds/1259) of `master-osx-64bit` 3completed successfully. |
| [20:07:12] | MythBuild: | Build [#746](https://code.mythtv.org/buildbot/#builders/156/builds/746) of `master-f32–64bit` 3completed successfully. |
| [20:07:46] | MythBuild: | Build [#1305](https://code.mythtv.org/buildbot/#builders/11/builds/1305) of `master-fedora-rawhide-64bit` 3completed successfully. |
| [20:08:05] | MythBuild: | Build [#1281](https://code.mythtv.org/buildbot/#builders/105/builds/1281) of `master-debian-buster-64bit` 3completed successfully. |
| [20:16:14] | MythBuild: | Build [#1054](https://code.mythtv.org/buildbot/#builders/112/builds/1054) of `master-debian-buster-rpi2-armv7l` 3completed successfully. |
| [20:40:09] | peterbennett: | bill6502: Thanks for updating it :) |
| [20:40:30] | bill6502: | np, thanks for writing it |
| [20:52:22] | DevMythNotifyBot: | COMMIT: kmdewaal pushed 1 commit(s) to branch master: https://github.com/MythTV/mythtv/compare/409e . . . ba9a7b3eef79 |
| [20:52:55] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
| [22:16:51] | toeb (toeb!~toeb@HSI-KBW-091-089-212-012.hsi2.kabel-badenwuerttemberg.de) has quit (Ping timeout: 272 seconds) | |
| [22:31:20] | markspieth2 (markspieth2!~markspiet@1.152.108.231) has joined #mythtv | |
| [22:31:20] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has joined #mythtv | |
| [22:31:20] | markspieth2 (markspieth2!~markspiet@1.152.108.231) has quit (Changing host) | |
| [22:31:20] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v markspieth2 | |
| [22:41:08] | DevMythNotifyBot (DevMythNotifyBot!~nodebot@cust32-dsl91-135-4.idnet.net) has quit (Read error: Connection reset by peer) | |
| [22:41:27] | DevMythNotifyBot (DevMythNotifyBot!~nodebot@cust32-dsl91-135-4.idnet.net) has joined #mythtv | |
| [22:45:38] | markspieth2 (markspieth2!~markspiet@202.169.118.74) has joined #mythtv | |
| [22:45:38] | markspieth2 (markspieth2!~markspiet@mythtv/developer/markspieth) has joined #mythtv | |
| [22:45:38] | markspieth2 (markspieth2!~markspiet@202.169.118.74) has quit (Changing host) | |
| [22:45:38] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v markspieth2 | |
| [23:41:09] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has joined #mythtv | |
| [23:41:09] | Mode for #mythtv by ChanServ!ChanServ@services.libera.chat : +v peterbennett | |
| [23:52:24] | peterbennett (peterbennett!~peter@mythtv/developer/peterbennett) has quit (Quit: Leaving.) | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.