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.

Sign up for GitHub

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

Jump to bottom

Move maxmind information log to debug and per-session #797

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Move maxmind information log to debug and per-session #797

merged 1 commit into from
Sep 21, 2023

Conversation

XAMPPRocky
Copy link
Collaborator

Previously this was being triggered on every packet after we moved the maxmind lookup to the start.

@XAMPPRocky XAMPPRocky enabled auto-merge (squash) September 21, 2023 08:49
@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 6387e8fe-c3e7-4e80-b52e-fc10cd1fd0fc

The following development images have been built, and will exist for the next 30 days:

To build this version:

git fetch git@github.com:googleforgames/quilkin.git pull/797/head:pr_797 && git checkout pr_797
cargo build

@XAMPPRocky XAMPPRocky merged commit 21fc1de into main Sep 21, 2023
@markmandel markmandel deleted the ep/mv-asn-log branch September 21, 2023 17:27
@markmandel markmandel added the kind/cleanup Refactoring code, fixing up documentation, etc label Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markmandel markmandel markmandel approved these changes

@No-ops No-ops Awaiting requested review from No-ops

@rezvaneh rezvaneh Awaiting requested review from rezvaneh

@vallfors vallfors Awaiting requested review from vallfors

Assignees
No one assigned
Labels
kind/cleanup Refactoring code, fixing up documentation, etc size/s
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@XAMPPRocky @quilkin-bot @markmandel