Ticket #11732 (closed: fixed)

Opened 5 years ago

Last modified 5 years ago

SCARF remote login through web service: fix issues with special characters in passwords

Reported by: Federico M Pouzols Owned by: Federico M Pouzols
Priority: critical Milestone: Release 3.5
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Roman Tolchenov

Description

It seems that there are issues with certain special characters used in user passwords. Clarify and fix if possible with the web service, otherwise some users may not be able to log in using the tomographic reconstruction custom interface.

Change History

comment:1 Changed 5 years ago by Nick Draper

  • Milestone changed from Release 3.4 to Release 3.5

comment:2 Changed 5 years ago by Federico Montesino Pouzols

  • Status changed from new to inprogress

uri-encode, now the auth script reqs escaped special chars, re #11732

Changeset: 6c0757276d9f99f01fd94aa804676555077a750c

comment:3 Changed 5 years ago by Federico Montesino Pouzols

do a safe pct-encode on login uri components, re #11732

Changeset: c4ef00dcee44a39bdbb003c768d47568bd6c6dc6

comment:4 Changed 5 years ago by Federico Montesino Pouzols

  • Status changed from inprogress to verify
  • Resolution set to fixed

This is being verified as pull request #759.

comment:5 Changed 5 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:6 Changed 5 years ago by Roman Tolchenov

A couple of things:

  1. the camelCase should be used (urlComponentEncode).
  2. the method should be static.

comment:7 Changed 5 years ago by Federico Montesino Pouzols

use camel case and static-ize urlEncode method, re #11732

Changeset: 50f4f206262a8acba01e863fb81ba13b1fa20d1a

comment:8 Changed 5 years ago by Federico Montesino Pouzols

Oops, thanks for noticing. It should be better now.

comment:9 Changed 5 years ago by Federico Montesino Pouzols

win7-docs-test failed on MDNormDirectSC-v1 because of unrelated issues, but in principle all builds/targets looked happy.

comment:10 Changed 5 years ago by Roman Tolchenov

  • Status changed from verifying to closed

Merge pull request #759 from mantidproject/11732_scarf_login_scarf_login_token.py_script_special_characters

Handle special characters in passwords - log into SCARF through web service

Full changeset: c811895a3bea12b95aac757cf5d41fa45eb9db3e

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 12570

Note: See TracTickets for help on using tickets.