www.cendio.com
Bug 5675 - Remove extproc.EncryptString
: Remove extproc.EncryptString
Status: CLOSED FIXED
: ThinLinc
VSM Server
: trunk
: PC Unknown
: P2 Normal
: 4.7.0
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2015-10-16 13:44 by
Modified: 2016-09-23 10:15 (History)


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description From cendio 2015-10-16 13:44:49
Nothing is using extproc.EncryptString apart from the "if __name__ ==
'__main__':" part of the extproc file, and an unused import into
handler_reqsession. It looks like it's made for openssl as well, so even if
we'd want it around for functionality we'd have to rewrite it. My suggestion is
to remove it altogether.
------- Comment #2 From cendio 2016-06-17 13:26:00 -------
Done.

There was nothing calling this, so there isn't really anything to test. Tester
could check the commit though and double-check that I didn't miss anything.
------- Comment #3 From cendio 2016-06-20 15:58:11 -------
(In reply to comment #2)
> Done.
> 
> There was nothing calling this, so there isn't really anything to test. Tester
> could check the commit though and double-check that I didn't miss anything.

Nothing left in the repo:

> $ git grep EncryptString; echo $?
> 1

Fine by me.