search

openalto / alto

Standard Application-Layer Traffic Optimization (ALTO) Toolset.
MIT License
2 stars 6 forks source link

Change architecture to separate parser logic from server logic (untested) #5

Open jacobdunefsky opened 2 years ago

jacobdunefsky commented 2 years ago

I separated the parser logic from the server logic (thank you Kai for his comment on #24 (https://github.com/openalto/ietf-hackathon/issues/24)). Now, the parser lives in a G2Parser class that is used in the server. Note that parser code can easily be swapped out, if we want to develop another parser, as long as the parser implements a method construct_from_flows(flows_list).

fno2010 commented 2 years ago

Overall, the PR looks good to me. But some files are still using hard TAB indents.