Uploaded image for project: 'Workterra BenAdmin'
  1. Workterra BenAdmin
  2. WB-504

Virgin | Server error on life plan, Pendin EOI- reject screen

    Details

    • Environment:
      Production
    • Categories:
      Enrollment - Life/LTD plans
    • Company:
      Virgin Atlantic
    • Reported by:
      Client
    • Stage Company Name:
      Virgin
    • Level:
      Admin

      Description

      Hi Team,
      Client faced a server error on Pending EOI screen, while rejecting the coverage amount. Please see the screen shot below on the email that I received.

      let me know if there is anything i can do to resolve it through UI.

      Getting members details from client. In the mean time could you please investigate and provide feedback on this server error.

      cc Christina Karnes Robyn Adkison

      apxSpecs

        Attachments

          Activity

          Hide
          vijayendra Vijayendra Shinde (Inactive) added a comment -

          Hi Monika Demla,

          Below are 3c's of reported issue-

          Concern: Server error on Pending EOI screen, while rejecting the coverage amount.

          Cause: We save rejection remark and user details who rejected changes in database. While saving user details, we save Firstname and lastname of admin who are doing change - in this case admin name is "Carrissa D'Anna". While saving this user name, query terminated in between due to signle quote present in Lastname of user (D'Anna). This caused server error while rejecting coverage amount.

          Correction: We need code changes to resolve this issue. SQL query parameterization is required. This will handle single quote in admin name properly. Record will be saved without any issue.

          Please let us know if you need more details on this.

          Thanks,
          Vijayendra

          Show
          vijayendra Vijayendra Shinde (Inactive) added a comment - Hi Monika Demla , Below are 3c's of reported issue- Concern: Server error on Pending EOI screen, while rejecting the coverage amount. Cause: We save rejection remark and user details who rejected changes in database. While saving user details, we save Firstname and lastname of admin who are doing change - in this case admin name is "Carrissa D'Anna". While saving this user name, query terminated in between due to signle quote present in Lastname of user (D'Anna). This caused server error while rejecting coverage amount. Correction: We need code changes to resolve this issue. SQL query parameterization is required. This will handle single quote in admin name properly. Record will be saved without any issue. Please let us know if you need more details on this. Thanks, Vijayendra
          Hide
          vijayendra Vijayendra Shinde (Inactive) added a comment -

          Hi Rohan J Khandave,

          We analyzed reported issue and found that query parameterization is required to handle single quote in username. Below is error log for reported issue-

          ErrorLog.txt

          Code file details are as below-

          Solution: Business Logic
          Project Name: Employee
          File Name: Employee.vb
          Function Name: SavePendingEOI_Component
          Line No.: 2632

          We need paramenterization of below mentioned query-
          RejectionQuery = "INSERT INTO Rejection_Details(EmployeeID,RejectionTypeID,UserName,Reason,Remarks,IsCompanyUSer,UserID,RoleID,Datetimestamp)" & _
          " VALUES(" & EmployeeID & "," & RejectionTypeID & ",'" & UserName & "','" & RejectionReason & "','" & Remarks & "',0," & I_userID & "," & RoleID & ",getdate())"

          Please confirm other queries as well who has username saved in database in accept or reject case so that this issue will not be reported in future.

          Thanks,
          Vijayendra
          CC:Amit Kumawat, Nidhi Kaul, Swapnil Pandhare

          Show
          vijayendra Vijayendra Shinde (Inactive) added a comment - Hi Rohan J Khandave , We analyzed reported issue and found that query parameterization is required to handle single quote in username. Below is error log for reported issue- ErrorLog.txt Code file details are as below- Solution: Business Logic Project Name: Employee File Name: Employee.vb Function Name: SavePendingEOI_Component Line No.: 2632 We need paramenterization of below mentioned query- RejectionQuery = "INSERT INTO Rejection_Details(EmployeeID,RejectionTypeID,UserName,Reason,Remarks,IsCompanyUSer,UserID,RoleID,Datetimestamp)" & _ " VALUES(" & EmployeeID & "," & RejectionTypeID & ",'" & UserName & "','" & RejectionReason & "','" & Remarks & "',0," & I_userID & "," & RoleID & ",getdate())" Please confirm other queries as well who has username saved in database in accept or reject case so that this issue will not be reported in future. Thanks, Vijayendra CC: Amit Kumawat , Nidhi Kaul , Swapnil Pandhare
          Hide
          Monika.Demla Monika Demla (Inactive) added a comment - - edited

          oh, this makes sense.. client is aware that her email address causes issues.. I will inform the client.
          In the mean time I will reject the pending EOI.

          Show
          Monika.Demla Monika Demla (Inactive) added a comment - - edited oh, this makes sense.. client is aware that her email address causes issues.. I will inform the client. In the mean time I will reject the pending EOI.
          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment -

          Hello Monika Demla,

          As mentioned in 3C's , this issue occurred only for those users having single quote in name. Which is very rare case.

          So can we change priority of ticket to Medium , please let us know your views on same.

          Note : This is not related to UI Refresh.

          Thanks,
          RohanK

          CC – Jennifer Reed , Satya , Jyoti Mayne ,Umesh Kadam

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - Hello Monika Demla , As mentioned in 3C's , this issue occurred only for those users having single quote in name. Which is very rare case. So can we change priority of ticket to Medium , please let us know your views on same. Note : This is not related to UI Refresh. Thanks, RohanK CC – Jennifer Reed , Satya , Jyoti Mayne , Umesh Kadam
          Hide
          jenniferreed Jennifer Reed (Inactive) added a comment -

          Rohan J Khandave Satya Rakesh Roy Thank you Rohan - don't we have a ticket created for quotes? We were planning to scan the entire system for quotes so we can identify any where we have inconsistent validation so that issues like these stop occuring.
          If we have not converted that ticket to WB can we do so?

          But yes I agree this should be a medium priority.
          Have we fixed everyone impacted for Monika?

          Show
          jenniferreed Jennifer Reed (Inactive) added a comment - Rohan J Khandave Satya Rakesh Roy Thank you Rohan - don't we have a ticket created for quotes? We were planning to scan the entire system for quotes so we can identify any where we have inconsistent validation so that issues like these stop occuring. If we have not converted that ticket to WB can we do so? But yes I agree this should be a medium priority. Have we fixed everyone impacted for Monika?
          Hide
          rohan.khandave Rohan J Khandave (Inactive) added a comment -

          Hello Jennifer Reed,

          Thanks for confirmation on changing priority. As per your comments we changed priority to medium.

          Have we fixed everyone impacted for Monika? : We did't fix any thing. But as per Monika Demla last comments , she is doing rejection for company.

          Thanks,
          RohanK

          CC – Satya , Rakesh Roy , Jyoti Mayne

          Show
          rohan.khandave Rohan J Khandave (Inactive) added a comment - Hello Jennifer Reed , Thanks for confirmation on changing priority. As per your comments we changed priority to medium. Have we fixed everyone impacted for Monika? : We did't fix any thing. But as per Monika Demla last comments , she is doing rejection for company. Thanks, RohanK CC – Satya , Rakesh Roy , Jyoti Mayne
          Hide
          Monika.Demla Monika Demla (Inactive) added a comment -

          This is complete, the rejection is processed as well.
          Thank you all.

          Show
          Monika.Demla Monika Demla (Inactive) added a comment - This is complete, the rejection is processed as well. Thank you all.
          Hide
          Monika.Demla Monika Demla (Inactive) added a comment -

          This is done and I am closing the ticket.
          cc Christina Karnes

          Show
          Monika.Demla Monika Demla (Inactive) added a comment - This is done and I am closing the ticket. cc Christina Karnes

            People

            Assignee:
            Monika.Demla Monika Demla (Inactive)
            Reporter:
            Monika.Demla Monika Demla (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 0h
                0h
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 5.5h
                5.5h