-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
AppEntry should be exported #780
Comments
|
Possibly. I will try to adapt our code for it. A mention of this would be good, on the migration document. |
Yes, I went with Bellow is the code change: pub fn configure(cfg: &mut web::RouterConfig, addr: Addr<actor::Machine>) {
cfg.data(API::new(addr))
.route("/info", web::get().to(API::info))
.route("/log", web::get().to(API::log))
.route("/probe", web::post().to(API::probe))
.route(
"/update/download/abort",
web::post().to(API::download_abort),
);
} |
I just ran in this exact same issue - it should probably be highlighted in |
Exact same issue. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
When splitting applications, it is expected to have an
app
or similar function which constructs the routes.For example:
is expected to work. However,
ApiEntry
is not exposed.It is used at:
actix-web/src/app.rs
Lines 41 to 56 in 7a28b32
So it'd make sense to have it exported.
The text was updated successfully, but these errors were encountered: