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

Large Update required #30

Open
JohnRoux opened this issue Feb 27, 2020 · 6 comments
Open

Large Update required #30

JohnRoux opened this issue Feb 27, 2020 · 6 comments

Comments

@JohnRoux
Copy link
Contributor

JohnRoux commented Feb 27, 2020

Hey,

Keen to help out on this package and do several improvements.

Let me know what you think about the following:

Routes

  • Add config option for route guards
  • Add config option for route existing
  • Add config option for route path

Logger

  • Change config option for driver to allow for class names, if the class is an instance of 'ApiLoggerInterface' then automatically allow it (this should allow for far easier overriding of the loggers
  • Add config option for when the response body should be saved (>400, >500 etc)
  • Add config option for a blacklist array for request variables (eg: password)

View

  • Add config options for which things to show on the view page
  • Add response body as an option

General

  • Fix the bug where the database table is created when not using that driver

Repo

  • Contribution Guidelines
  • Build out a Wiki
@AbbyJanke
Copy link

Pull Request #25 has two options for a config option of blacklist array.

@AbbyJanke
Copy link

An option to disable the web interface would also be nice.

@aungwinthant
Copy link
Owner

Sounds great. We can release a major version after this update. Please submit PR for the changes. I will merge as soon as possible. Sorry for late replies, previously I was busy with work stuffs.

@JohnRoux
Copy link
Contributor Author

Will get started on this early next week 👍

@JohnRoux
Copy link
Contributor Author

Sorry :(
Things have been crazy this side.
It's on my backlog, but not sure when I'm going to have free (non-client) dev time available in the next couple weeks 👍

@dansleboby
Copy link
Contributor

I made a few of the things in #42

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

No branches or pull requests

4 participants