Uploaded image for project: 'Ambari'
  1. Ambari
  2. AMBARI-12476

Widget response differs when using ambarisessionid and basic auth

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.1.0
    • 2.3.0, 2.1.2
    • ambari-server
    • None

    Description

      Got different responses when using basic auth like "curl -u admin:admin
      and when using ambarisessionid in cookie. With basic auth default cluster widgets do not show in /api/v1/clusters/c1/widgets

      Attachments

        1. AMBARI-12476.patch
          15 kB
          Dmytro Sen
        2. AMBARI-12476.trunk.patch
          5 kB
          Siddharth Wagle
        3. AMBARI-12476-2.1.2.patch
          19 kB
          Siddharth Wagle

        Issue Links

          Activity

            hadoopqa Hadoop QA added a comment -

            +1 overall. Here are the results of testing the latest attachment
            http://issues.apache.org/jira/secure/attachment/12746349/AMBARI-12476.patch
            against trunk revision .

            +1 @author. The patch does not contain any @author tags.

            +1 tests included. The patch appears to include 1 new or modified test files.

            +1 javac. The applied patch does not increase the total number of javac compiler warnings.

            +1 release audit. The applied patch does not increase the total number of release audit warnings.

            +1 core tests. The patch passed unit tests in ambari-server.

            Test results: https://builds.apache.org/job/Ambari-trunk-test-patch/3433//testReport/
            Console output: https://builds.apache.org/job/Ambari-trunk-test-patch/3433//console

            This message is automatically generated.

            hadoopqa Hadoop QA added a comment - +1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12746349/AMBARI-12476.patch against trunk revision . +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in ambari-server. Test results: https://builds.apache.org/job/Ambari-trunk-test-patch/3433//testReport/ Console output: https://builds.apache.org/job/Ambari-trunk-test-patch/3433//console This message is automatically generated.
            dsen Dmytro Sen added a comment -

            Committed to trunk

            dsen Dmytro Sen added a comment - Committed to trunk
            hudson Hudson added a comment -

            SUCCESS: Integrated in Ambari-trunk-Commit #3182 (See https://builds.apache.org/job/Ambari-trunk-Commit/3182/)
            AMBARI-12476 Widget response differs when using ambarisessionid and basic auth (dsen) (dsen: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=3411efbc55017c4947ee494faaa05270aaecee4f)

            • ambari-server/src/main/java/org/apache/ambari/server/api/services/WidgetService.java
            • ambari-server/src/main/java/org/apache/ambari/server/orm/dao/WidgetDAO.java
            • ambari-server/src/test/java/org/apache/ambari/server/controller/internal/WidgetResourceProviderTest.java
            • ambari-server/src/main/java/org/apache/ambari/server/orm/entities/WidgetEntity.java
            • ambari-server/src/main/java/org/apache/ambari/server/controller/internal/WidgetResourceProvider.java
            hudson Hudson added a comment - SUCCESS: Integrated in Ambari-trunk-Commit #3182 (See https://builds.apache.org/job/Ambari-trunk-Commit/3182/ ) AMBARI-12476 Widget response differs when using ambarisessionid and basic auth (dsen) (dsen: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=3411efbc55017c4947ee494faaa05270aaecee4f ) ambari-server/src/main/java/org/apache/ambari/server/api/services/WidgetService.java ambari-server/src/main/java/org/apache/ambari/server/orm/dao/WidgetDAO.java ambari-server/src/test/java/org/apache/ambari/server/controller/internal/WidgetResourceProviderTest.java ambari-server/src/main/java/org/apache/ambari/server/orm/entities/WidgetEntity.java ambari-server/src/main/java/org/apache/ambari/server/controller/internal/WidgetResourceProvider.java

            Issue still present with WidgetLayoutService.

            swagle Siddharth Wagle added a comment - Issue still present with WidgetLayoutService.

            Current patch + WidgetLayoutService fix.

            swagle Siddharth Wagle added a comment - Current patch + WidgetLayoutService fix.
            sumitmohanty Sumit Mohanty added a comment -

            +1 to backport the change

            sumitmohanty Sumit Mohanty added a comment - +1 to backport the change

            Pushed patch to branch-2.1 and differential patch to trunk.

            swagle Siddharth Wagle added a comment - Pushed patch to branch-2.1 and differential patch to trunk.
            hudson Hudson added a comment -

            FAILURE: Integrated in Ambari-branch-2.1 #460 (See https://builds.apache.org/job/Ambari-branch-2.1/460/)
            AMBARI-12476. Widget response differs when using ambarisessionid and basic auth. (swagle: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=5cbfb2a6318d96dd1facb4ec081c9a35134a260e)

            • ambari-server/src/main/java/org/apache/ambari/server/orm/dao/WidgetDAO.java
            • ambari-server/src/main/java/org/apache/ambari/server/controller/internal/WidgetResourceProvider.java
            • ambari-server/src/main/java/org/apache/ambari/server/api/services/WidgetService.java
            • ambari-server/src/main/java/org/apache/ambari/server/api/services/WidgetLayoutService.java
            • ambari-server/src/test/java/org/apache/ambari/server/controller/internal/WidgetResourceProviderTest.java
            • ambari-server/src/main/java/org/apache/ambari/server/orm/entities/WidgetEntity.java
            hudson Hudson added a comment - FAILURE: Integrated in Ambari-branch-2.1 #460 (See https://builds.apache.org/job/Ambari-branch-2.1/460/ ) AMBARI-12476 . Widget response differs when using ambarisessionid and basic auth. (swagle: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=5cbfb2a6318d96dd1facb4ec081c9a35134a260e ) ambari-server/src/main/java/org/apache/ambari/server/orm/dao/WidgetDAO.java ambari-server/src/main/java/org/apache/ambari/server/controller/internal/WidgetResourceProvider.java ambari-server/src/main/java/org/apache/ambari/server/api/services/WidgetService.java ambari-server/src/main/java/org/apache/ambari/server/api/services/WidgetLayoutService.java ambari-server/src/test/java/org/apache/ambari/server/controller/internal/WidgetResourceProviderTest.java ambari-server/src/main/java/org/apache/ambari/server/orm/entities/WidgetEntity.java
            hudson Hudson added a comment -

            SUCCESS: Integrated in Ambari-trunk-Commit #3367 (See https://builds.apache.org/job/Ambari-trunk-Commit/3367/)
            AMBARI-12476. Widget response differs when using ambarisessionid and basic auth. (swagle: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=25b0f3ea2d36ead842990dc73de28155f708d233)

            • ambari-server/src/main/java/org/apache/ambari/server/api/services/WidgetLayoutService.java
            hudson Hudson added a comment - SUCCESS: Integrated in Ambari-trunk-Commit #3367 (See https://builds.apache.org/job/Ambari-trunk-Commit/3367/ ) AMBARI-12476 . Widget response differs when using ambarisessionid and basic auth. (swagle: http://git-wip-us.apache.org/repos/asf?p=ambari.git&a=commit&h=25b0f3ea2d36ead842990dc73de28155f708d233 ) ambari-server/src/main/java/org/apache/ambari/server/api/services/WidgetLayoutService.java

            People

              dsen Dmytro Sen
              dsen Dmytro Sen
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: