チャンネルスキャンが既存のエンドポイント/オプションで動作しなくなっている。

[hiradimir]

Environment

• Version of Mirakurun: 3.4.0
• Version of Node: v14.15.3
• Version of NPM: 6.14.9
• Platform:
  • Linux raspberrypi 5.4.79-v7l+ #1373 SMP Mon Nov 23 13:27:40 GMT 2020 armv7l GNU/Linux
• Status of Mirakurun:
  • https://gist.github.com/hiradimir/d7f3eaa5bac3b8a7086240cf1f824a5e

Issue

チャンネルスキャンが既存のエンドポイント/オプションで動作しなくなっていました。

$ sudo curl -X PUT "http://localhost:40772/api/config/channels/scan"
channel scanning... (type: "GR")

-> total 0 channels (of Takeover is 0) found and 46 channels stored.

channel scan has completed and saved successfully.
**RESTART REQUIRED** to apply changes.

#79 の PR が契機と思われます。

挙動としてはminCh/maxChを指定すると動作するため、

$ sudo curl -X PUT "http://localhost:40772/api/config/channels/scan?type=GR"
channel scanning... (type: "GR")

-> total 0 channels (of Takeover is 0) found and 46 channels stored.

channel scan has completed and saved successfully.
**RESTART REQUIRED** to apply changes.
$ sudo curl -X PUT "http://localhost:40772/api/config/channels/scan?type=GR&minCh=13&maxCh=67"
channel scanning... (type: "GR")

channel: "13" ...
-> 3 services found.
-> {"name":"ＮＨＫＥテレ","type":"GR","channel":"13"}

channel: "14" ...

以下で判定が想定通りで無いように見えました。

Mirakurun/src/Mirakurun/api/config/channels/scan.ts

Lines 66 to 68 in a8674a1

	case common.ChannelTypes.GR:
	option.startCh = option.startCh === undefined ? 13 : option.startCh;
	option.endCh = option.endCh === undefined ? 62 : option.endCh;

---

以下は推測です。

API定義上のデフォルト値も消えている。

Mirakurun/src/Mirakurun/api/config/channels/scan.ts

Lines 304 to 315 in a8674a1

	{
	in: "query",
	name: "minCh",
	type: "integer",
	description: "Specifies the minimum number of channel numbers to scan."
	},
	{
	in: "query",
	name: "maxCh",
	type: "integer",
	description: "Specifies the maximum number of channel numbers to scan."
	},

fw? middleware? に何を使っているのか把握出来ていないのですが、
GRとかのデフォルト値が扱われているところをみると、
type: "integer" で、 0 に丸められる挙動になってしまっているのでは無いでしょうか。

[kanreisa]

ご報告ありがとうございます。バグです。
@3.4.1 で修正 (c7466ff) したのでご確認いただけますでしょうか？

[hiradimir]

@kanreisa
対応ありがとうございます。
@3.4.1 にて解消されていることが確認できました。