Pharo Issue Tracker - Manuscript (Case [Issue]22886) Zinc - Zinc should allow custom server string

Manuscript Notification

	Sven Van Caekenberghe resolved Case 22886: Zinc should allow custom server string and assigned it to Torsten Bergmann: Enhancement in Zinc: 1. Pharo Image • You are subscribed to this case

I understand the issue, but I do not agree with the fact that this is a must fix bug for Pharo 7, far from it.

It is relatively easy to overwrite the default Server header on a per server instance granularity today. And that is what you want, not a global default. Consider:

<code>
ZnServer startDefaultOn: 8080.

ZnServer default delegate in: [ :delegate |
ZnServer default delegate: (ZnValueDelegate with: [ :request | | response |
response := delegate handleRequest: request.
response headers at: #Server put: 'Google'.
response ]) ].

(ZnClient new get: ZnServer default localUrl; response) headers at: #Server.
</code>

The above wraps any existing delegate, lets the original delegate do its work, and then modifies the response.

I also think very few people will ever need to make this change. Like you said, it is part of production hardening.

For me, Enhancement, Would be nice, Pharo 8

Category changed:
Bug

Enhancement

Milestone changed:
Pharo7.0

Pharo8.0

Priority changed:
3 – Must Fix

4 – Would be nice

	Priority: 4 – Would be nice		Status: Resolved (Fix Review Needed)
	Assigned to: Torsten Bergmann		Milestone: Pharo8.0

Go to Case

No longer need updates? Unsubscribe from this case.

Don't want Manuscript notifications anymore? Update your preferences.

_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
https://lists.gforge.inria.fr/mailman/listinfo/pharo-bugtracker