Uploaded image for project: 'New Features 2017'
  1. New Features 2017
  2. NF-2334

All Company- Employee Login - URL parameters - Security - URL parameters in all the SSM pages,reports are displayed in plain text.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: High
    • Resolution: Bug Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: UI Refresh
    • Labels:
      None
    • Environment:
      Pre Production
    • Bug Type:
      Functional
    • Bug Severity:
      Medium
    • Level:
      Employee
    • Module:
      BenAdmin - Security
    • Reported by:
      Harbinger
    • Company:
      All Clients/Multiple Clients
    • Item State:
      Stage QA - Production Deployment on Hold
    • Issue Importance:
      Q2

      Description

      All Company- Employee Login - URL parameters - Security - URL parameter values in all the SSM pages,reports are displayed in plain text.

      As observed all the URL parameter values are displayed in plain english text and can be vulnerable for security breach.

      This can be generic issue and may exist for Admin,Partners,SA user roles too.

      CC : Vijayendra ShindeSachin HingoleRakesh RoyHrishikesh DeshpandeRohan J KhandaveSamir

        Attachments

        1. CCSF_URLData_notEncrypted.jpg
          CCSF_URLData_notEncrypted.jpg
          222 kB
        2. EmpBen.jpg
          EmpBen.jpg
          99 kB
        3. ParameterURL.jpg
          ParameterURL.jpg
          180 kB

          Issue Links

            Activity

            prasadp Prasad Pise (Inactive) created issue -
            vijayendra Vijayendra Shinde (Inactive) made changes -
            Field Original Value New Value
            Status Open [ 1 ] In Development [ 10007 ]
            Hide
            vijayendra Vijayendra Shinde (Inactive) added a comment -

            hi Prasad Pise,

            When you verified on Azure, Load test was in progress. During load test we make encryption false due to which all parameters shown in plain text.

            Now on Azure parameters are encrypted.

            Thanks,
            Vijayendra

            Show
            vijayendra Vijayendra Shinde (Inactive) added a comment - hi Prasad Pise , When you verified on Azure, Load test was in progress. During load test we make encryption false due to which all parameters shown in plain text. Now on Azure parameters are encrypted. Thanks, Vijayendra
            vijayendra Vijayendra Shinde (Inactive) made changes -
            Assignee Vijayendra Shinde [ ID10506 ] Prasad Pise [ prasadp ]
            Resolution System Behaviour [ 10100 ]
            Status In Development [ 10007 ] Rejected [ 10004 ]
            prasadp Prasad Pise (Inactive) made changes -
            Link This issue relates to NF-2714 [ NF-2714 ]
            prasadp Prasad Pise (Inactive) made changes -
            Status Rejected [ 10004 ] Closed [ 6 ]
            prasadp Prasad Pise (Inactive) made changes -
            Module Parent values: BenAdmin(10100) Parent values: BenAdmin(10100)Level 1 values: Security(10112)
            Show
            prasadp Prasad Pise (Inactive) added a comment - - edited HI Vijayendra Shinde I have observed some more URLs as mentioned below which has parameters in plain English text. Could you please confirm. 1. All the URLs of 'SetPageTrackingValue' Some Examples as follows : https://preprod.workterra.net/Platform/PTHomePage/SetPageTrackingValue?PageName=Manage+Beneficiaries&URL=%2FBenAdmin%2FUserDetails%2FUserDetails%2FEmployeeBeneficiary&ProjectID=2&ModuleID=2&_=1515748444932 https://preprod.workterra.net/Platform/PTHomePage/SetPageTrackingValue?PageName=Initiate+Qualifying+Event&URL=%2FPlatform%2FUserDetails%2FUserDetails%2FInitiateQualifyingEvents&ProjectID=2&ModuleID=2&_=1515759719648 https://preprod.workterra.net/Platform/PTHomePage/SetPageTrackingValue?PageName=&URL=%2FBenAdmin%2FReport%2FReport%2FEnrollmentReport&ProjectID=2&ModuleID=2&_=1515760436680 2. URLs displayed on mouse hover. PFA screenshot. 3. View Paycheck url https://preprod.workterra.net/BenAdmin/UserDetails/UserDetails/ViewCompare?PaycheckID=232&ViewClick=1&Popup=1 3. Any Reports URL https://preprod.workterra.net/Platform/Reserved.ReportViewerWebControl.axd?OpType=Resource&Version=11.0.2802.16&Name=ViewerScript CC Rakesh Roy Samir Nidhi Kaul Satya Gaurav Sodani
            prasadp Prasad Pise (Inactive) made changes -
            Status Closed [ 6 ] Reopen in Production [ 10027 ]
            prasadp Prasad Pise (Inactive) made changes -
            Assignee Prasad Pise [ prasadp ] Vijayendra Shinde [ ID10506 ]
            prasadp Prasad Pise (Inactive) made changes -
            Attachment ParameterURL.jpg [ 71722 ]
            prasadp Prasad Pise (Inactive) made changes -
            Bug Severity Medium [ 16702 ]
            Company All Clients/Multiple Clients [ 18434 ]
            Environment Pre Production [ 18470 ]
            Priority Highest [ 1 ] High [ 2 ]
            pratap.patil Pratap Patil (Inactive) made changes -
            Assignee Vijayendra Shinde [ ID10506 ] Pratap Patil [ pratap.patil ]
            pratap.patil Pratap Patil (Inactive) made changes -
            Status Reopen in Production [ 10027 ] In Development [ 10007 ]
            pratap.patil Pratap Patil (Inactive) made changes -
            Item State Parent values: Development(10200)Level 1 values: In Progress(10206)
            Show
            prasadp Prasad Pise (Inactive) added a comment - Hi Vijayendra Shinde As discussed, please refer the following URL https://preprod.workterra.net/Platform/Customization/Customization/PreviewChangeApproval?CallFromPageID=8&EffectiveDate=3/1/2018%2012:00:00%20AM&IsFromConfigureChangeApproval=False&Popup=1 HTTP/1.1
            pratap.patil Pratap Patil (Inactive) logged work - 19/Feb/18 12:37 PM
            • Time Spent:
              5h
               

              Analysis, Code Correction and unit testing , impact testing .

            pratap.patil Pratap Patil (Inactive) made changes -
            Code Reviewed By Vijayendra Shinde [ 11901 ]
            pratap.patil Pratap Patil (Inactive) made changes -
            Item State Parent values: Development(10200)Level 1 values: In Progress(10206) Parent values: Development(10200)Level 1 values: Ready for Local Testing(10209)
            Hide
            pratap.patil Pratap Patil (Inactive) added a comment - - edited

            Hello Prasad Pise ,

            As discussed with you and Vijayendra Shinde , Changing the values of moduleId from SetPageTrackingValue method will not break the security . We are not able to reproduce the Report point mentioned in above comment . Rest other below points are fixed :

            • 'PreviewChangeApproval' URL is corrected , in this URL 'PopUp' parameter is not being used so that one is not encrypted.
            • URL on mouse over from screenshot also corrected but here name of module is not encrypted , because this is used as Enum values , so changing the module name will not create any impact on system .
            • 'ViewPayCheck' URL is corrected .

            These changes will be deployed in next CodeMap build .

            Thanks,
            Pratap Patil

            CC: Vijayendra Shinde , Sachin Hingole

            Show
            pratap.patil Pratap Patil (Inactive) added a comment - - edited Hello Prasad Pise , As discussed with you and Vijayendra Shinde , Changing the values of moduleId from SetPageTrackingValue method will not break the security . We are not able to reproduce the Report point mentioned in above comment . Rest other below points are fixed : 'PreviewChangeApproval' URL is corrected , in this URL 'PopUp' parameter is not being used so that one is not encrypted. URL on mouse over from screenshot also corrected but here name of module is not encrypted , because this is used as Enum values , so changing the module name will not create any impact on system . 'ViewPayCheck' URL is corrected . These changes will be deployed in next CodeMap build . Thanks, Pratap Patil CC: Vijayendra Shinde , Sachin Hingole
            pratap.patil Pratap Patil (Inactive) made changes -
            Assignee Pratap Patil [ pratap.patil ] Prasad Pise [ prasadp ]
            khandu.kshirsagar Khandu Kshirsagar (Inactive) made changes -
            Item State Parent values: Development(10200)Level 1 values: Ready for Local Testing(10209) Parent values: LB QA(10201)Level 1 values: LB Deployed(11600)
            pratap.patil Pratap Patil (Inactive) made changes -
            Remaining Estimate 0h [ 0 ]
            Time Spent 5h [ 18000 ]
            Worklog Id 105454 [ 105454 ]
            prasadp Prasad Pise (Inactive) logged work - 21/Feb/18 02:16 PM
            • Time Spent:
              2h
               

              Codemap Verification
              Internal Discussion

            prasadp Prasad Pise (Inactive) made changes -
            Status In Development [ 10007 ] Local Testing [ 10200 ]
            prasadp Prasad Pise (Inactive) made changes -
            Item State Parent values: LB QA(10201)Level 1 values: LB Deployed(11600) Parent values: LB QA(10201)Level 1 values: In Testing(10210)
            Show
            prasadp Prasad Pise (Inactive) added a comment - Hi Pratap Patil I am in testing for the fixed URLs, however as discussed, could you please check the following URLs also. 1. PCP Pop up scenario https://10.0.2.71/BenAdmin/UserDetails/UserDetails/PCPCodePopup?PlanDesignId=565&EffectiveDate=1/1/2018%2012:00:00%20AM&IsAlreadyEnrolled=&Popup=1 2. Edit/Update Employee Beneficiary request https://10.0.2.71/BenAdmin/UserDetails/UserDetails/EmployeePlanBeneficiary?PlanDesignID=249&EffectiveDate=1/1/2018%2012:00:00%20AM&PlanName=Basic%20Life/%20AD%26D%20(Full%20Coverage%20-%20$500000)%20-%20Basic%20Life&ProviderLogo=/Assets/Images/Company/13680/Benefit%20Provider/Sun_Life_Financial_logo.jpg&IsSpousePrimaryBeneficiary=0&IsBeneficiaryRequired=1&UserAction=edit&PrimaryList=%5B%7B%22RelationshipId%22:%221~16965%22,%22Perecentage%22:%22100%22,%22BeneficiaryId%22:%2276513%22,%22MemberID%22:%2216965%22,%22RelationshipName%22:%22%22,%22SpouseRelationshipId%22:%221%22%7D%5D&PlanIndex=0&_=1519304457734 Add another relation ship for beneficiary https://10.0.2.71/BenAdmin/UserDetails/UserDetails/OtherBeneficiaryPV?RelationId=8&EffectiveDate=1/1/2018%2012:00:00%20AM&RelationName=Business%20Associate&Beneficiarytype=Primary&PlanId=249&Index=1&Popup=1
            prasadp Prasad Pise (Inactive) logged work - 22/Feb/18 01:31 PM
            • Time Spent:
              1h
               

              ReTest
              Internal Discussion

            prasadp Prasad Pise (Inactive) made changes -
            Time Spent 5h [ 18000 ] 6h [ 21600 ]
            Worklog Id 105996 [ 105996 ]
            khandu.kshirsagar Khandu Kshirsagar (Inactive) made changes -
            Item State Parent values: LB QA(10201)Level 1 values: In Testing(10210) Parent values: Stage QA(10202)Level 1 values: Stage Deployed(11602)
            sachin.hingole Sachin Hingole (Inactive) made changes -
            Status Local Testing [ 10200 ] Stage Testing [ 10201 ]
            sachin.hingole Sachin Hingole (Inactive) made changes -
            Item State Parent values: Stage QA(10202)Level 1 values: Stage Deployed(11602) Parent values: Stage QA(10202)Level 1 values: In Testing(10214)
            prasadp Prasad Pise (Inactive) logged work - 23/Feb/18 12:54 PM
            • Time Spent:
              0.5h
               

              Internal Discussion
              URL repro

            prasadp Prasad Pise (Inactive) made changes -
            Time Spent 6h [ 21600 ] 6.5h [ 23400 ]
            Worklog Id 106091 [ 106091 ]
            komal.barde Komal Barde (Inactive) logged work - 23/Feb/18 05:22 PM
            • Time Spent:
              7h
               

              Analysis
              Code Correction
              Unit testing
              Discussion with Pratap and Prasad for issue repro

            komal.barde Komal Barde (Inactive) made changes -
            Time Spent 6.5h [ 23400 ] 13.5h [ 48600 ]
            Worklog Id 106153 [ 106153 ]
            Hide
            komal.barde Komal Barde (Inactive) added a comment -

            Hi Prasad Pise,

            I have committed changes related to below pages and it will get deployed in next build:

            1. PCP Pop up scenario
            2. Edit/Update Employee Beneficiary request
            3. Add another relation ship for beneficiary

            Thanks,
            Komal.

            Show
            komal.barde Komal Barde (Inactive) added a comment - Hi Prasad Pise , I have committed changes related to below pages and it will get deployed in next build: 1. PCP Pop up scenario 2. Edit/Update Employee Beneficiary request 3. Add another relation ship for beneficiary Thanks, Komal.
            Hide
            komal.barde Komal Barde (Inactive) added a comment -

            List of modified files:

            • /branches/UiRefresh-LB/Web/Web Projects/BenAdmin/Areas/UserDetails/Controllers/EmployeeBeneficiary/EmployeeBeneficiaryController.cs
            • /branches/UiRefresh-LB/Web/Web Projects/BenAdmin/Areas/UserDetails/Controllers/EnrollNow/EnrollNowController.cs
            • /branches/UiRefresh-LB/Web/Web Projects/BenAdmin/Areas/UserDetails/Models/EnrollNow/EnrollNowPVModel.cs
            • /branches/UiRefresh-LB/Web/Web Projects/BenAdmin/Areas/UserDetails/Views/UserDetails/EmployeeBeneficiary/EmployeeBeneficiary.cshtml
            Show
            komal.barde Komal Barde (Inactive) added a comment - List of modified files : /branches/UiRefresh-LB/Web/Web Projects/BenAdmin/Areas/UserDetails/Controllers/EmployeeBeneficiary/EmployeeBeneficiaryController.cs /branches/UiRefresh-LB/Web/Web Projects/BenAdmin/Areas/UserDetails/Controllers/EnrollNow/EnrollNowController.cs /branches/UiRefresh-LB/Web/Web Projects/BenAdmin/Areas/UserDetails/Models/EnrollNow/EnrollNowPVModel.cs /branches/UiRefresh-LB/Web/Web Projects/BenAdmin/Areas/UserDetails/Views/UserDetails/EmployeeBeneficiary/EmployeeBeneficiary.cshtml
            khandu.kshirsagar Khandu Kshirsagar (Inactive) made changes -
            Item State Parent values: Stage QA(10202)Level 1 values: In Testing(10214) Parent values: LB QA(10201)Level 1 values: LB Deployed(11600)
            komal.barde Komal Barde (Inactive) logged work - 26/Feb/18 12:10 PM
            • Time Spent:
              2h
               
              • Unit testing
              • Attained code review
              • Code merge and commit
            komal.barde Komal Barde (Inactive) made changes -
            Time Spent 13.5h [ 48600 ] 15.5h [ 55800 ]
            Worklog Id 106279 [ 106279 ]
            prasadp Prasad Pise (Inactive) made changes -
            Time Spent 15.5h [ 55800 ] 17.5h [ 63000 ]
            Worklog Id 106333 [ 106333 ]
            prasadp Prasad Pise (Inactive) made changes -
            Item State Parent values: LB QA(10201)Level 1 values: LB Deployed(11600) Parent values: LB QA(10201)Level 1 values: In Testing(10210)
            khandu.kshirsagar Khandu Kshirsagar (Inactive) made changes -
            Item State Parent values: LB QA(10201)Level 1 values: In Testing(10210) Parent values: Stage QA(10202)Level 1 values: Stage Deployed(11602)
            prasadp Prasad Pise (Inactive) made changes -
            Attachment EmpBen.jpg [ 73844 ]
            Hide
            prasadp Prasad Pise (Inactive) added a comment -

            Hi Komal Barde

            I have verified the fixes for following changes on Codemap
            1. PCP Pop up scenario
            2. Edit/Update Employee Beneficiary request
            3. Add another relation ship for beneficiary

            I have observed following issue:
            On Beneficiary page, Relationship Name's encrypted value is getting displayed on UI.
            This issue occurs when employee try to Add/Edit the beneficiary from employee beneficiary page.
            PFA screenshot.

            CC Vijayendra Shinde

            Show
            prasadp Prasad Pise (Inactive) added a comment - Hi Komal Barde I have verified the fixes for following changes on Codemap 1. PCP Pop up scenario 2. Edit/Update Employee Beneficiary request 3. Add another relation ship for beneficiary I have observed following issue: On Beneficiary page, Relationship Name's encrypted value is getting displayed on UI. This issue occurs when employee try to Add/Edit the beneficiary from employee beneficiary page. PFA screenshot. CC Vijayendra Shinde
            prasadp Prasad Pise (Inactive) logged work - 27/Feb/18 01:53 PM
            • Time Spent:
              2h
               

              Verification, Internal Discusison
              Issue Repro on Codemap and Preprod

            prasadp Prasad Pise (Inactive) made changes -
            Time Spent 17.5h [ 63000 ] 19.5h [ 70200 ]
            Worklog Id 106484 [ 106484 ]
            prasadp Prasad Pise (Inactive) made changes -
            Assignee Prasad Pise [ prasadp ] Komal Barde [ komal.barde ]
            komal.barde Komal Barde (Inactive) logged work - 27/Feb/18 04:57 PM
            • Time Spent:
              2h
               
              • Analysis
              • Code Correction
              • Unit testing
              • Discussion with Pratap and Prasad for issue repro
            sachin.hingole Sachin Hingole (Inactive) made changes -
            Resolution System Behaviour [ 10100 ] Unresolved [ 10200 ]
            Status Stage Testing [ 10201 ] Reopen in Stage [ 10023 ]
            sachin.hingole Sachin Hingole (Inactive) made changes -
            Item State Parent values: Stage QA(10202)Level 1 values: Stage Deployed(11602) Parent values: Stage QA(10202)Level 1 values: Re-open(10216)
            khandu.kshirsagar Khandu Kshirsagar (Inactive) made changes -
            Item State Parent values: Stage QA(10202)Level 1 values: Re-open(10216) Parent values: LB QA(10201)Level 1 values: LB Deployed(11600)
            Hide
            pratap.patil Pratap Patil (Inactive) added a comment -

            Hello Prasad Pise ,

            The beneficiary relationship name issue is fixed and deployed on CodeMap . Please verify it .

            Thanks,
            Pratap

            Show
            pratap.patil Pratap Patil (Inactive) added a comment - Hello Prasad Pise , The beneficiary relationship name issue is fixed and deployed on CodeMap . Please verify it . Thanks, Pratap
            pratap.patil Pratap Patil (Inactive) made changes -
            Assignee Komal Barde [ komal.barde ] Prasad Pise [ prasadp ]
            Hide
            prasadp Prasad Pise (Inactive) added a comment -

            HI Pratap Patil

            Beneficiary relationship name issue is resolved on Codemap. As observed, the relationship name is displayed in plain english text on Beneficiary Page and Beneficiary reports.

            Thanks

            • Prasad
            Show
            prasadp Prasad Pise (Inactive) added a comment - HI Pratap Patil Beneficiary relationship name issue is resolved on Codemap. As observed, the relationship name is displayed in plain english text on Beneficiary Page and Beneficiary reports. Thanks Prasad
            prasadp Prasad Pise (Inactive) made changes -
            Item State Parent values: LB QA(10201)Level 1 values: LB Deployed(11600) Parent values: LB QA(10201)Level 1 values: In Testing(10210)
            prasadp Prasad Pise (Inactive) made changes -
            Item State Parent values: LB QA(10201)Level 1 values: In Testing(10210) Parent values: LB QA(10201)Level 1 values: Ready for Stage(10213)
            khandu.kshirsagar Khandu Kshirsagar (Inactive) made changes -
            Item State Parent values: LB QA(10201)Level 1 values: Ready for Stage(10213) Parent values: Stage QA(10202)Level 1 values: Stage Deployed(11602)
            prasadp Prasad Pise (Inactive) made changes -
            Status Reopen in Stage [ 10023 ] In Development [ 10007 ]
            prasadp Prasad Pise (Inactive) made changes -
            Status In Development [ 10007 ] Local Testing [ 10200 ]
            prasadp Prasad Pise (Inactive) made changes -
            Status Local Testing [ 10200 ] Stage Testing [ 10201 ]
            prasadp Prasad Pise (Inactive) made changes -
            Item State Parent values: Stage QA(10202)Level 1 values: Stage Deployed(11602) Parent values: Stage QA(10202)Level 1 values: In Testing(10214)
            prasadp Prasad Pise (Inactive) logged work - 28/Feb/18 12:12 PM
            • Time Spent:
              3h
               

              Internal Discussions
              Verification on Codemap and Preprod environment for Beneficiary Changes
              Employee Self Serve Mode verification for Codemap and PreProd

            Hide
            prasadp Prasad Pise (Inactive) added a comment -

            HI Pratap Patil
            Beneficiary relationship name issue is resolved on PreProd environment. Now,Relationship name is displayed in plain english text on Beneficiary Page and Beneficiary reports.

            Thanks
            -Prasad

            Show
            prasadp Prasad Pise (Inactive) added a comment - HI Pratap Patil Beneficiary relationship name issue is resolved on PreProd environment. Now,Relationship name is displayed in plain english text on Beneficiary Page and Beneficiary reports. Thanks -Prasad
            prasadp Prasad Pise (Inactive) made changes -
            Item State Parent values: Stage QA(10202)Level 1 values: In Testing(10214) Parent values: Stage QA(10202)Level 1 values: Production Deployment on Hold(10224)
            prasadp Prasad Pise (Inactive) made changes -
            Time Spent 19.5h [ 70200 ] 22.5h [ 81000 ]
            Worklog Id 106674 [ 106674 ]
            komal.barde Komal Barde (Inactive) made changes -
            Time Spent 22.5h [ 81000 ] 24.5h [ 88200 ]
            Worklog Id 106781 [ 106781 ]
            prasadp Prasad Pise (Inactive) made changes -
            Status Stage Testing [ 10201 ] Production Testing [ 10202 ]
            prasadp Prasad Pise (Inactive) logged work - 20/Mar/18 12:28 PM
            • Time Spent:
              2h
               
              <No comment>
            prasadp Prasad Pise (Inactive) made changes -
            Resolution Unresolved [ 10200 ] Bug Fixed [ 10402 ]
            Status Production Testing [ 10202 ] Production Complete [ 10028 ]
            prasadp Prasad Pise (Inactive) made changes -
            Time Spent 24.5h [ 88200 ] 26.5h [ 95400 ]
            Worklog Id 109208 [ 109208 ]
            prasadp Prasad Pise (Inactive) made changes -
            Status Production Complete [ 10028 ] Closed [ 6 ]
            vijayendra Vijayendra Shinde (Inactive) made changes -
            Link This issue relates to DEV-13718 [ DEV-13718 ]
            Transition Time In Source Status Execution Times
            Vijayendra Shinde (Inactive) made transition -
            Open In Development
            3d 16h 42m 1
            Vijayendra Shinde (Inactive) made transition -
            In Development Rejected
            1m 41s 1
            Prasad Pise (Inactive) made transition -
            Rejected Closed
            162d 1h 38m 1
            Prasad Pise (Inactive) made transition -
            Closed Reopen in Production
            59d 6h 31m 1
            Pratap Patil (Inactive) made transition -
            Reopen in Production In Development
            37d 20h 39m 1
            Sachin Hingole (Inactive) made transition -
            Stage Testing Reopened in Stage
            4d 23h 5m 1
            Prasad Pise (Inactive) made transition -
            Reopened in Stage In Development
            5h 19m 1
            Prasad Pise (Inactive) made transition -
            In Development In LB Testing
            3d 2
            Prasad Pise (Inactive) made transition -
            In LB Testing Stage Testing
            21h 38m 2
            Prasad Pise (Inactive) made transition -
            Stage Testing In Production Testing
            14d 18h 40m 1
            Prasad Pise (Inactive) made transition -
            In Production Testing Production Complete
            6d 5h 48m 1
            Prasad Pise (Inactive) made transition -
            Production Complete Closed
            1m 30s 1

              People

              Assignee:
              prasadp Prasad Pise (Inactive)
              Reporter:
              prasadp Prasad Pise (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 26.5h
                  26.5h