Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-3859

Improve error message for invalid certificates

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Waiting for testing
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.9.5
    • Fix Version/s: None
    • Component/s: Security
    • Labels:
    • Testing Instructions:
      Hide

      This issue needs to be tested in Android.

      Please wait until MOBILE-3860 is integrated before testing this issue because it can affect this behaviour.

      1. In the app, enter the URL of a site with an invalid certificate (check comments to see a site like this).
      2. Click to connect. Check that you see an error saying that the certificate isn't valid and you should contact the site admin.
      Show
      This issue needs to be tested in Android. Please wait until  MOBILE-3860 is integrated before testing this issue because it can affect this behaviour. In the app, enter the URL of a site with an invalid certificate (check comments to see a site like this). Click to connect. Check that you see an error saying that the certificate isn't valid and you should contact the site admin.
    • Affected Branches:
      MOODLE_39_STABLE
    • Pull Master Branch:
    • Sprint:
      Moodle App 4.0

      Description

      In the 3.9.5 version of the app we started using advanced-http plugin to do network requests in Android, to fix CORS issues with the Ionic WebView plugin. It seems this native request is more strict with certificates than WebView requests, we're recieving some complains about not being able to connect after the update.

      The plugin supports disabling certificate checks, but we cannot do that in the app since it can cause security issues. So IMO we should improve the error message in this case so the user understands why the app cannot connect.

       

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              dpalou Dani Palou
              Reporter:
              dpalou Dani Palou
              Peer reviewer:
              Pau Ferrer Pau Ferrer
              Integrator:
              Pau Ferrer Pau Ferrer
              Tester:
              Isabel Renedo Rouco Isabel Renedo Rouco
              Participants:
              Component watchers:
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated: