Skip to content

feat: implementation of scala fibonacci sample#483

Closed
octonato wants to merge 1 commit intomainfrom
rgc/implement-fibonacci-scala
Closed

feat: implementation of scala fibonacci sample#483
octonato wants to merge 1 commit intomainfrom
rgc/implement-fibonacci-scala

Conversation

@octonato
Copy link
Copy Markdown
Member

Not working yet hence draft.
I tried to run service locally and it's not responding. So, probably some issue when converting?

@octonato octonato force-pushed the rgc/implement-fibonacci-scala branch from dc2d3fd to a32dd46 Compare September 23, 2021 14:59
Copy link
Copy Markdown
Contributor

@johanandren johanandren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No proto file?

With both the proxy and your application running, any defined endpoints should be available at `http://localhost:9000`. In addition to the defined gRPC interface, each method has a corresponding HTTP endpoint. Unless configured otherwise (see [Transcoding HTTP](https://docs.lbcs.dev/js-services/proto.html#_transcoding_http)), this endpoint accepts POST requests at the path `/[package].[entity name]/[method]`. For example, using `curl`:

```shell
curl -XPOST -H "Content-Type: application/json" localhost:9000/fibonacci.Fibonacci/NextNumber -d '{"value": 5 }'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be /com.example.fibonacci.Fibonnaci/NextNumber right?

@@ -0,0 +1,70 @@
# Fibonacci Action Service

This project is based on the Akka Serverless Maven archetype.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is all copied and not correct for the Scala version

@RayRoestenburg
Copy link
Copy Markdown
Contributor

Continuing here: #517

@octonato octonato deleted the rgc/implement-fibonacci-scala branch October 12, 2021 19:46
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.

3 participants