python-bottleでエラー500がでる

Question

スマートフォンでLEDを光らすor消すがしたい

ラズベリーパイゼロWを用いてiotしたいので、簡単にできるツールを探した結果bottleにたどりつき、HelloWorldはできました。
しかし、JavaScriptの実装やボタンの配置などをしようとコードを書いて表示させようとしたときに以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

n
127.0.0.1 - - [29/Aug/2019 15:46:08] "GET / HTTP/1.1" 200 291
favicon.ico
Traceback (most recenr call last):
  File "/usr/lib/python2.7/dist-packages/bottle.py", line 862, in _handle return route.call(**args)
  File "/usr/lib/python2.7/dist-packages/bottle.py", line 1740, in wrapper rv = callback(*a, **ka)
  File "/home/pi/Documents/controlLED_on_web.py", line 37, in index led_num = int(led)
ValueError: invalid literal for int() with base 19: 'favicon.ico'
127.0.0.1 - - [29/Aug/2019 15:46:12] "GET /favicon.ico HTTP/1.1" 500 745

該当のソースコード

from bottle import route, run
import RPi.GPIO as GPIO

host = '192.168.1.8'

GPIO.setmode(GPIO.BCM)
led_pins = [2, 4, 5]
led_states = [0, 0, 0]
switch_pin = 17

GPIO.setup(led_pins[0], GPIO.OUT)
GPIO.setup(led_pins[1], GPIO.OUT)
GPIO.setup(led_pins[2], GPIO.OUT)
GPIO.setup(switch_pin, GPIO.IN, pull_up_down=GPIO.PUD_UP)

def switch_status():
    state = GPIO.input(switch_pin)
    if state:
        return 'Up'
    else:
        return 'Down'

def html_for_led(led):
    l = str(led)
    result = " <input type='button' onClick='changed(" + l + ")' value='LED " + l + "'/>"
    return result

def update_leds():
    for i, value in enumerate(led_states):
        GPIO.output(led_pins[i], value)

@route('/')
@route('/<led>')
def index(led="n"):
    print(led)
    if led != "n":
        led_num = int(led)
        led_states[led_num] = not led_states[led_num]
        update_leds()
    response = "<script>"
    response += "function changed(led)"
    response += "{"
    response += "  window.location.href='/' + led"
    response += "}"
    response += "</script>"
    
    response += '<h1>GPIO Control</h1>'
    response += '<h2>Button=' + switch_status() + '</h2>'
    response += '<h2>LEDs</h2>'
    response += html_for_led(0) 
    response += html_for_led(1) 
    response += html_for_led(2) 
    return response

run(host='0.0.0.0', port=80)

試したこと

favicon.icoと書いてあったため　https://www.favicon.cc/?action=icon&file_id=14586　のアイコンをダウンロードし上記の@route('/')の下に下記のコードを書いてみたりしました（変数の型が違うみたいなことを言われたためやめた）。どうにかしてfavicon.icoを送れればいいと思ってますがあってますか？

@route('/favicon.ico')
def favcon():
  fav=None
  return fav('favicon.ico', root='./static')

またこのサイトを見て.icoが送れそうなところを抜粋して書いたりしました。

補足情報（FW/ツールのバージョンなど）

使っているラズベリーパイはRasPizeroWでアプリはPython3(IDLE)です。
なお、python-bottleのバージョンが古いのかと思いコマンドでインストールするやつをやったところ最新のやつでした。

Answer 1

現在 index()関数の実装は

@route('/')
@route('/<led>')
def index(led="n"):
    print(led)
    if led != "n":
        led_num = int(led)
        # (以下略)

となっておりますので、'/favicon.ico' が呼ばれた場合 led='favicon.ico' の引数で index関数が呼ばれてしまいます。

その場合は、led != 'n' の条件文は Trueとなりますので、その後の処理が行われ、最終的に

led_num = int(led)

にて、文字列 'favicon.ico' が int型にキャストできずにエラーが発生してしまいます。

とりあえずの修正としては上記のコードを

@route('/')
@route('/<led:int>')
def index(led=None):
    print(led)
    if (led is not None) and (0 <= led < len(led_status)):
        led_num = int(led)
        # (以下略)

のようにすることで

• @route('/<led:int>') とすることで、int型以外の引数で index()関数が呼ばれないようにする
• 引数がない事を明確にするために、デフォルト引数を led=None に変更する（これは好み）
• ledの数の範囲外の値で呼ばれた場合の条件文のなかに入らないようにする

できるのではないでしょうか。

Comments

[退会済みユーザー]

<led>にfavicon.icoが代入されてしまうということでよろしいでしょうか？
あまり関係ないですが is not None　という構文があるんですね、、
python全く知らなくてこのようなところに頼ってしまったので

上記の該当のコードを直して試したところ404 Not foundとなってしまいました。
エラーコードです。
```
None
127.0.0.1 - - [29/Aug/2019 18:40:47] "GET / HTTP/1.1" 200 291
127.0.0.1 - - [29/Aug/2019 18:40:53] "GET /favicon.ico HTTP/1.1" 404 735
```

[magichan]

favicon.ico はお気に入り登録されたときなどに、そこに表示されるアイコンですので、別に無くても問題ありませんし、実際に無いのであれば 404 になるのは妥当かと思います。
「ブラウザが要求してきたけど、それは無いよ」くらいの意味なので気にする必要も無い気がします。

[magichan]

それとも、あえて　favicon.ico を返すような仕様にしたいということでしょうか？

[退会済みユーザー]

すいません。「それはないよ～」でいいのですがやりかたがわかりません。よろしければ教えてください。
回答いただきありがとうございます。

[退会済みユーザー]

すみません。読み間違えてました。404になって結果画面が表示されないので、どうにかできませんか？たびたびすみません。。

[magichan]

ん？
「404 Not found」になるのは、あくまでも "http://****/favicon.ico " を呼んだときかと思うのですが、違います？？
通常通り、 "http://*****/ " を呼んだ場合や、"http://*****/1 " などと数値の場合は問題なく index関数が呼ばれるので従来どおりの動作をすると思いますよ。

[退会済みユーザー]

できてました、、、
http://***/Noneでできるのかなぁ～って思ってできなくて勘違いしてました、、申し訳ないです。
この度はありがとうございました！