Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#18879 closed enhancement (worksforme)

[Patch] Allow to opt-out from JOSM's certificate amendment

Reported by: simon04 Owned by: simon04
Priority: normal Milestone:
Component: Core Version:
Keywords: tls certificates yourkit Cc:

Description


Attachments (1)

18879.patch (974 bytes ) - added by simon04 4 years ago.

Download all attachments as: .zip

Change History (8)

by simon04, 4 years ago

Attachment: 18879.patch added

comment:1 by simon04, 4 years ago

Keywords: yourkit added

Previously on the heap:

sun.security.provider.X509Factory 1_084_648
Last edited 4 years ago by simon04 (previous) (diff)

comment:2 by stoecker, 4 years ago

Please give a reason why this should be possible.

comment:3 by simon04, 4 years ago

Reduced initialization time, reduced memory consumption, reduced complexity (for users not needed the supplementary certificates), reduced attack surface (for suspicious users).

in reply to:  3 comment:4 by stoecker, 4 years ago

Replying to simon04:

Reduced initialization time, reduced memory consumption

Small compared to immense amount of stuff we do

reduced complexity (for users not needed the supplementary certificates)

An opt-out icreased the complexity - it does not reduce it.

, reduced attack surface (for suspicious users).

We don't decide about certs ourself. We only override the slowness of the Java world to follow the rest of the world.

If I weight the minimal advantages against the disadvantages we will have to deal with when users fail to load perfectly valid imagery I don't see much benefit in this.

comment:5 by Don-vip, 4 years ago

It is already possible if you run JOSM with --offline=certificates. What do you want more?

comment:6 by Don-vip, 4 years ago

Milestone: 20.03
Resolution: worksforme
Status: assignedclosed

And we have also this property since r9995:

    /**
     * Add missing root certificates to the list of trusted certificates for TLS connections.
     * @throws IOException if an I/O error occurs
     * @throws GeneralSecurityException if a security error occurs
     */
    public static void addMissingCertificates() throws IOException, GeneralSecurityException {
        if (!Config.getPref().getBoolean("tls.add-missing-certificates", true))
            return;
Last edited 4 years ago by Don-vip (previous) (diff)

comment:7 by simon04, 4 years ago

I'm stupid, it was just in front of my eyes, but I overlooked it. Thanks for the clarification, Vincent!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain simon04.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.