Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix for issue 911 and 966 #1173

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

xjh33445566
Copy link

I just do some simple code to fullfill the feature request.
Hope this code can do some help for you.

@xjh33445566 xjh33445566 changed the title fix for issue 911 fix for issue 911 and 966 May 31, 2020
@lepe
Copy link

lepe commented Aug 11, 2022

PR merged into/solved in the unofficial build - Release 2+

@AN3Orik
Copy link

AN3Orik commented Aug 12, 2023

Sorry, but this is bull**it. Why json type by default? That's broke any template engine integration out-of-box.

@AN3Orik
Copy link

AN3Orik commented Aug 12, 2023

This commit also broke logic where application already have content-type redefentions for POST requests. For example: jQuery ajax parse result depend on Content-Type received from controller. + default auto detection content type by extension making to use this fork impossible for projects where spark used more than just for hold REST requests.

Not to mention about sneaky Configuration outside Spark class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants