alternative server_method api
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 25/100
Direção de pesquisa
Comece rastreando o comportamento existente de server_methods e como as declarações em app/hyperstack/models e app/models são carregadas no cliente e no servidor. Defina o escopo de remote_access_to, dos blocos de segurança compartilhados, das definições exclusivas do servidor e de no_cache; em seguida, verifique se o comportamento existente dos métodos do servidor permanece intacto.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
given you have an existing model, which needs some of the methods to be defined as server_methods, it would be nice to say something like:
remote_access_to(:foo, :bar) { acting_user.admin? }
where foo and bar are existing instance methods.
or
remote_access_to(:foo, bar: 12) ...
which will give a default value of 12 to bar.
This is nice not only because it allows for existing methods to be declared as server methods without modification, but it also allows for a common security block to be provided, instead of repeating it. It also is a nicer separation of concerns.
Basic implementation is straight forward: On the client methods :foo and :bar are defined like any other server method, and will call the method with the security name prefix. On the server, the secured method calls the block, and if successfully returns true, continues on and calls the method.
There are complexities:
-
if the remote_access_to declaration is placed BEFORE the definition of foo and bar, you have to make sure that those definitions are server only. It might be possible by redefining define_method, to check for this case...
-
If foo and bar are server side only, then it would be nice if they could be declared in a file that is in app/models. This can be done by putting the server and client defs in
app/hyperstack/models, and then putting server only defs inapp/modelsand adding arequire Rails.root.join('app', 'models', 'sample.rb') unless RUBY_ENGINE == 'opal'at the start of the file, and maybe with some hacking we could make a method calledrequire_server_definitionsthat would automatically figure out the file being loaded, and then load the same file from app/models.
And a nice feature:
server_methods by default assume that they are returning a value, and thus will cache the value on the client once its retrieved. You can force a re-retrieval of the value by adding a ! to the method name. But in many cases you just want the method to be run on the server every time its called, anyway. So how about a no_cache parameter added to the server method args.
- Linguagem predominante
- JavaScript
- Estrelas
- 538
- Forks
- 41
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de hyperstack-org/hyperstack
-
expose_as_operations method Abertaenhancement needs doc
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
hyperstack-org/hyperstack#359 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
hyperstack-org/hyperstack#267 ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
hyperstack-org/hyperstack#457 ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
hyperstack-org/hyperstack#456 ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 15/100
hyperstack-org/hyperstack#455 · 2 comentários ·
Todas as issues de hyperstack-org/hyperstack
Issues semelhantes
-
bug customer-eng Durable Agents Inngest status: needs triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
-
optimization optimization:agents-md-curator
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
githubnext/gh-aw-cao#13475 ·
-
[BUG]: "Clear All" in Settings doesn't clear the saved analysis, old data comes back after reload Abertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
AOSSIE-Org/OrgExplorer#253 · 1 comentário ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
oxc-project/oxc#26944 ·
-
ai-observability bug team/ai-observability
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100