Bug 5675 - Remove extproc.EncryptString
Summary: Remove extproc.EncryptString
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: VSM Server (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.7.0
Assignee: Pierre Ossman
URL:
Keywords: derfian_tester, prosaic
Depends on:
Blocks:
 
Reported: 2015-10-16 13:44 CEST by Karl Mikaelsson
Modified: 2016-09-23 10:15 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:


Attachments

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.

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