dronrathore / goexpress Goto Github PK

I just tried the hello world from the readme and go throws this error

# command-line-arguments
./main.go:8:16: cannot use func literal (type func(*goexpress.Request, *goexpress.Response)) as type goexpress.Middleware in argument to app.Get
./main.go:9:8: res.Write undefined (type *goexpress.Response is pointer to interface, not interface)

Hi, I was wondering if it is possible to use this library to render an HTML view file at a given path. And if so, how?

Add a boundary header and push a file to the host.

Add Gzip/tar for downloadable file and SendFile.
Could be done through helpers like

app.Set('file-compress', 'gzip')
app.Set('download-compress', 'tar')

This will also require to share the app Params map, a pointer of same can be shared with response objects.

Currently goexpress doesn't support On fly request updation, clients sometimes divide the request in multiple parts and hence require a 100 Continue response.

As #18 is under dev and there are been so many fixes pushed since v1.0.2, I am wondering if after completing #18 we bump the version to v2.0.0 as in no way older versions would be compatible. Though I will keep the recent code in a new headless branch and will keep on pushing bug fixes to it.

Let me know your thoughts. 😄

Currently Body doesn't populate on boundary params based requests.
Fix: Parse and Populate Body for form-data.

In Line #61 you log out every request to console. It would be better to use a logging middleware like morgan in ExpressJS. What do you think?

Add async+sync event based Stream piping functionality similar to nodejs. This can be a separate project too that can be pulled in this with import.

Middlewares shouldn't be counted as a route match as this leads to 404 too as a positive 200 response, as we are pushing middlewares in routes array itself we need to add a key of type in Route struct and then cross check that in express.go file to decide whether to increase the route match counter or not.

This will be a breaking change!

I am planning to change the directory structure of this project, currently you need to require all the packages individually to use them(yeah I wrote this very badly back then).

I am going to merge them under goexpress itself. This will make things more easy as:

• You need to require only one package
• Overall code of the repo too will improve

Please leave your suggestions/ideas below.

Add an app param to enable/disable logging of incoming request and actions.

Hi, good project! I like ExpressJS in Node.

Do you plan to add your solution to julienschmidt's benchmark test? It would be good to see the performance of goexpress.

I follow your project and maybe add some PRs too :)

So dron i have seen this project quite a while ago and was really amused of it, I m a 15 y/o kid from india and m really interested to carry on this project, i think ur busy with your professional work, if u dont mind can I carry on this project, i have lots of time spare and lots of things to learn

Need to refactor this repo, sorry for being a slacker and not maintaining it for quite while, refactor will add support for context, and coding styles to make it more Go like(obviously a breaking change).

Looking forward to the Christmas holidays to finish the refactor. If you want some must to have features, feel free to add them here.