Project

General

Profile

Defect #423

URLの最後に/があるとNG?

Added by Haru Iida about 10 years ago. Updated about 10 years ago.

Status:
終了(Closed)
Priority:
通常(Normal)
Assignee:
-
Category:
RedmineREST-API
Target version:
Start date:
06/05/2010
Due date:
% Done:

100%

Estimated time:

Description

ローカルでテストしたところRedmine URLが

http://localhost:3000

の場合は上手くチケット情報を取れますが

http://localhost:3000/

と最後に/が付いているとチケット情報を取得できませんでした。

Redmine側には以下のようなエラーがログに出力されていました。

[2010-06-05 16:21:19] ERROR bad URI `//issues.xml?assigned_to_id=me&key=19875f6e84d3f50d8078fa2b3ef4ff5d1cd2515b'.

Related issues

Related to RedmineAir - Feature #457: URLのValidation終了(Closed)Akiko Takano06/22/2010

Actions
#1

Updated by yusuke kokubo about 10 years ago

入力されたURLに単純に/issues.xmlをつけてるので、もともとのURLに/がついてると
/が2つあるのでas3の標準ライブラリ的にエラーになるみたいですね。
修正するのは末尾に/がついてたら削除すればよいかと思いますが、そんな単純でよいかと思うとちょっと心配。

#2

Updated by Akiko Takano about 10 years ago

さきほど私も確認しました。
ただし、localhostでのテストです。
ActionScriptの問題ではないようで、Redmine(Rails)側が、/projects と //projects を別物として処理しています。
(/コントローラ/アクション/...というお約束なので)
IEとかFirefoxで直接上記のURLを叩くとStatus Code 400で、同じエラーになりますね。

サーバ側がApacheやmongrelとかでプロキシしてある場合はどうなるか、試してみないといけないですけれど、おそらく "//" は "/"に変換するといいんじゃないでしょうか。

あと、下記は実行時のデバッグのメッセージです。

Error #2044: Unhandled ioError:. text=Error #2032: Stream Error. URL: http://localhost:3000//issues.xml?watcher_id=me&key=6a7fbc787585340b45a75ef7f403f263d5baa908
    at redmineair/load()
    at redmineair/___redmineair_Button2_click()

#3

Updated by yusuke kokubo about 10 years ago

  • Category set to RedmineREST-API
  • Target version set to 0.0.1

ActionScriptの問題ではないようで、Redmine(Rails)側が、/projects と //projects を別物として処理しています。

Redmine側の問題なんですね。

サーバ側がApacheやmongrelとかでプロキシしてある場合はどうなるか、試してみないといけないですけれど、おそらく "//" は "/"に変換するといいんじゃないでしょうか。

その方法がシンプルで良いかも。
間違える人も多そうなので早めに直した方が良いですね。

#4

Updated by Akiko Takano about 10 years ago

可能なら、URLとして入力した文字列のValidationがあるといいかも。
package com.redmineAir.util 以下に、データの加工やvaludation用のクラスがあると良いかなと思っています。

#5

Updated by Akiko Takano about 10 years ago

  • Status changed from 新規(New) to 担当(Assigned)
  • % Done changed from 0 to 80

rev. 31で対応してみました。
もうちょっと綺麗なやり方があると思いますが...。

#6

Updated by Akiko Takano about 10 years ago

  • Status changed from 担当(Assigned) to 終了(Closed)
  • % Done changed from 80 to 100

#457 と合わせて、終了とします。

Also available in: Atom PDF