Uploaded image for project: 'Teeda'
  1. Teeda
  2. TEEDA-306

サブミット連打対策が不十分だったのを修正

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.7
    • Fix Version/s: 1.0.8
    • Component/s: None
    • Labels:
      None

      Description

      サブミットを連打したときに、これまでは、同じロジックが再度実行されることはないが、タイミングによっては、正しい画面に遷移しないときや、リダイレクトの循環実行とみなされてしまうことがあった。

        Activity

        Hide
        higa higa added a comment -

        修正しました。ただし、リダイレクトの循環実行は、フレームワーク側では、サブミット連打と区別つかないため、チェックをはずしました。これからは、アプリケーション側でチェックする必要があります。

        Show
        higa higa added a comment - 修正しました。ただし、リダイレクトの循環実行は、フレームワーク側では、サブミット連打と区別つかないため、チェックをはずしました。これからは、アプリケーション側でチェックする必要があります。
        Hide
        jflute jflute added a comment -

        > 正しい画面に遷移しないとき

        これは具体的にどういった例外になりますでしょうか?

        例えばTomcatだと

        Caused by: java.lang.IllegalStateException
        at org.apache.catalina.connector.ResponseFacade.sendRedirect(ResponseFacade.java:432)

        という感じでしょうか?

        > リダイレクトの循環実行

        同じく
        これは具体的にどういった例外になりますでしょうか?

        Caused by: org.seasar.teeda.core.exception.AlreadyRedirectingException: [ETDA0029]既にリダイレクトされています。
        at org.seasar.teeda.extension.util.TeedaExtensionErrorPageManagerImpl.handleException(TeedaExtensionErrorPageManagerImpl.java:74)
        at org.seasar.teeda.core.lifecycle.LifecycleImpl.handleException(LifecycleImpl.java:112)

        という感じでしょうか?

        利用者でこの現象で悩んでいる方もいると思うので、例外を具体的に示唆した方が良いかと思いました。
        また、既にリリース済みでVersionをUpできないような利用者のために、
        どういった修正をしたのかを具体的に提示するのも大事かと思います。
        (というか自分がその状況だったり...)

        > これからは、アプリケーション側でチェックする必要があります。

        こちら、1.0.7から1.0.8へ移行する際の注意点というような形で
        利用者へ通知した方がいいかと思いました。(リリースメール等で)

        Show
        jflute jflute added a comment - > 正しい画面に遷移しないとき これは具体的にどういった例外になりますでしょうか? 例えばTomcatだと Caused by: java.lang.IllegalStateException at org.apache.catalina.connector.ResponseFacade.sendRedirect(ResponseFacade.java:432) という感じでしょうか? > リダイレクトの循環実行 同じく これは具体的にどういった例外になりますでしょうか? Caused by: org.seasar.teeda.core.exception.AlreadyRedirectingException: [ETDA0029] 既にリダイレクトされています。 at org.seasar.teeda.extension.util.TeedaExtensionErrorPageManagerImpl.handleException(TeedaExtensionErrorPageManagerImpl.java:74) at org.seasar.teeda.core.lifecycle.LifecycleImpl.handleException(LifecycleImpl.java:112) という感じでしょうか? 利用者でこの現象で悩んでいる方もいると思うので、例外を具体的に示唆した方が良いかと思いました。 また、既にリリース済みでVersionをUpできないような利用者のために、 どういった修正をしたのかを具体的に提示するのも大事かと思います。 (というか自分がその状況だったり...) > これからは、アプリケーション側でチェックする必要があります。 こちら、1.0.7から1.0.8へ移行する際の注意点というような形で 利用者へ通知した方がいいかと思いました。(リリースメール等で)

          People

          • Assignee:
            higa higa
            Reporter:
            higa higa
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development