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

Display blocked IP in logs #24

Merged
merged 1 commit into from
Sep 22, 2020
Merged

Conversation

akunzai
Copy link
Contributor

@akunzai akunzai commented Sep 22, 2020

No description provided.

@jissereitsma jissereitsma merged commit 0a29172 into MageBridge:master Sep 22, 2020
@jissereitsma
Copy link
Collaborator

@akunzai Thanks for contributing. Unfortunately, my effort to do anything (!) with MageBridge has dropped. I'm no longer maintaining any Joomla project, I'm no longer maintaining MageBridge. And Magento 1 is outdated and unsupported.

If you are using these sources, please consider creating your own fork instead and using these sources. Because from the sources under this specific repository, no packages will be build. It's just source code, waiting for someone to take over maintenance by forking it.

@jissereitsma
Copy link
Collaborator

An additional note: If there PRs coming in that only take a fraction of time, I'm going to merge them. But as soon as more work is needed, I'm inclined to close the PR without reviewing. Having your own fork allows you to make any change as you see fit.

@akunzai akunzai deleted the log-blocked-ip branch September 22, 2020 07:30
@akunzai
Copy link
Contributor Author

akunzai commented Sep 22, 2020

An additional note: If there PRs coming in that only take a fraction of time, I'm going to merge them. But as soon as more work is needed, I'm inclined to close the PR without reviewing. Having your own fork allows you to make any change as you see fit.

Hi @jissereitsma ,I will create my own fork, Thanks for suggestions.

simbus82 added a commit to simbus82/MageBridgeCore that referenced this pull request Sep 29, 2020
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.

2 participants