Bug 5675

Summary:	Remove extproc.EncryptString
Product:	ThinLinc	Reporter:	Karl Mikaelsson <derfian>
Component:	VSM Server	Assignee:	Pierre Ossman <ossman>
Status:	CLOSED FIXED
Severity:	Normal	CC:	ossman
Priority:	P2	Keywords:	derfian_tester, prosaic
Version:	trunk
Target Milestone:	4.7.0
Hardware:	PC
OS:	Unknown
Acceptance Criteria:

Description Karl Mikaelsson cendio

2015-10-16 13:44:49 CEST

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 Pierre Ossman cendio

2016-06-17 13:26:00 CEST

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 Karl Mikaelsson cendio

2016-06-20 15:58:11 CEST

(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.