fix xml parser read outside bounds #1844

This fix made the parser survive more than 100 million fuzzer iterations, so I think it works.
https://github.com/davisking/dlib/issues/1844

still not fixed


Limit number of candidates?

In some files, completion menus take up the whole page, which I never want. Is there a way to limit the number of candidates shown?
Basically, g:deoplete#max_list for NCM.

Why not setting the pumheight option?

Yes, @Alok , you can simply set pumheight=20 (change 20 to whatever you like), then the popup menu will not occupy the full screen. See :h 'pumheight'.


Prosody NixOS tests time out

Describe the bug
The NixOS tests for prosody and prosody-mysql time out on hydra, unnecessarily delaying the channel by about 9 hours.
Steps To Reproduce
e.g.
https://hydra.nixos.org/build/199758102
https://hydra.nixos.org/build/199758852
Expected behavior
Tests should finish promptly or fail, not stall the channel.
Notify maintainers
@Izorkin @NinjaTrappeur
Metadata
Happens on both nixos-22.11 and nixos-unstable.

I'm on it.

https://github.com/NixOS/nixpkgs/pull/202491


I used Avalonia for the first time, but it didn't work on Linux

I compiled the program from windows and uploaded it to CentOS 7.
After running, nothing happened.
The window didn't render, and there was no error message.
The light just stopped blinking on the command line

I prefixed each line with Console. Writeline();
all lines work well
but no window was rendered
windows works well but linux
What's the reason, please

How are you publishing your app before uploading?
What is your XOrg version?
Are you running the app from a terminal emulator running inside X11 session?

How are you publishing your app before uploading?
What is your XOrg version?
Are you running the app from a terminal emulator running inside X11 session?

Thank you for your reply
I use VS2019 publish it.  (release |AnyCPU;.net core 3.0;linux x64)
XOrg is GNOME 3.28
running by terminal emulator

https://github.com/AvaloniaUI/Avalonia/issues/3679 has similar symptoms.
Please, check if following code prints anything on console:
        Console.WriteLine(SKTypeface.Default.FamilyName);
        Console.WriteLine(SKTypeface.FromFamilyName("Sans").FamilyName);

#3679 has similar symptoms.
Please, check if following code prints anything on console:
        Console.WriteLine(SKTypeface.Default.FamilyName);
        Console.WriteLine(SKTypeface.FromFamilyName("Sans").FamilyName);

thank you for your reply
now, it throw an NullRefrenceException at Console.WriteLine(SKTypeface.FromFamilyName("Sans").FamilyName)

#3679也有类似的症状。
请检查以下代码是否打印出控制台上的任何内容：
        Console.WriteLine(SKTypeface.Default.FamilyName);
        Console.WriteLine(SKTypeface.FromFamilyName("Sans").FamilyName);

thanks so much！
I set the window‘s default FontFamily and solved it!
It's so cool!!
I hope that one day Avalonia can be the no.1 of  cross plantform UI framework solutions!!!!!!!
<Application.Resources>
        <FontFamily x:Key="yh">msyh</FontFamily>
</Application.Resources>
<Style Selector="Window">
            <Setter Property="FontFamily" Value="{StaticResource yh}" />
</Style>

Are you using nightly build or a current release?


Fix warnings and errors on Julia 0.5 (replaces #113)

@carlobaldassi, I took the liberty to edit the two commits I wanted
changes in according to my line comments on #113, in order to get this
done before my vacation ends tomorrow. I hope you don't mind :)
Thanks for helping me with this - I really appreciate it!
Closes #113

Don't mind at all of course, sorry for not doing it myself, I hadn't had the time. Thanks a lot!


Problem if wrong password entered after reset

Changed my Okta password, and as expected the gimme-aws-creds call first failed, then prompted me for a new password.
I entered the old password by accident, which got written to the Keychain gimme-aws-creds entry.
This resulted in an error I couldn't get around until I updated the Keychain entry with the proper password (I suppose I could have removed the entry from the Keychain as well).
Expected Behavior

Ideally, when the password fails, it should always prompt for the password to be entered, and it shouldn't write the password to the Keychain until it has successfully logged in.
Current Behavior

When login fails after password change, the login fails. User must run gimme-aws-creds a second time to get the password prompt. There is a question asking if this should be written to the Keychain, but the password is not validated prior to the write leaving the client in a bad state.
Possible Solution

Don't write the password to keychain until after it succeeds.
And probably always prompt for the password when the one in the Keychain fails.
Steps to Reproduce (for bugs)

Change the password in the Keychain to something bogus OR change your Okta password
Run gimme-aws-creds
If prompted for password, give it a bad password
Once the bad password is in the Keychain, you should see something like:

Using password from keyring for<EMAIL_ADDRESS>Traceback (most recent call last):
  File "/usr/local/bin/gimme-aws-creds", line 17, in <module>
    GimmeAWSCreds().run()
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/main.py", line 447, in run
    self._run()
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/main.py", line 738, in _run
    for data in self.iter_selected_aws_credentials():
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/main.py", line 714, in iter_selected_aws_credentials
    for role in self.aws_selected_roles:
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/main.py", line 632, in aws_selected_roles
    selected_roles = self._get_selected_roles(self.requested_roles, self.aws_roles)
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/main.py", line 623, in aws_roles
    self.saml_data['SAMLResponse'],
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/main.py", line 614, in saml_data
    self._cache['saml_data'] = saml_data = self.okta.get_saml_response(self.aws_app['links']['appLink'])
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/main.py", line 607, in aws_app
    self._cache['aws_app'] = aws_app = self._get_selected_app(self.conf_dict.get('aws_appname'), self.aws_results)
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/main.py", line 560, in aws_results
    self.okta.auth_session()
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/okta.py", line 156, in auth_session
    login_response = self.auth()
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/okta.py", line 145, in auth
    flow_state = self._login_username_password(None, self._okta_org_url + '/api/v1/authn')
  File "/usr/local/lib/python3.7/site-packages/gimme_aws_creds/okta.py", line 336, in _login_username_password
    response.raise_for_status()
  File "/usr/local/lib/python3.7/site-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 401 Client Error: Unauthorized for url: https://workfront.okta.com/api/v1/authn```

<!-- And so on -->

## Context
<!--- How has this issue affected you? What are you trying to accomplish? -->
<!--- Providing context helps us come up with a solution that is most useful in the real world -->

## Your Environment
<!--- Include as many relevant details about the environment you experienced the bug in -->
* App Version used: gimme-aws-creds 2.2.0 (was also using 2.1.0 prior to having this issue)
* Environment name and version:
* Operating System and version: Mac OS X

This not only applies to incorrectly typed saved passwords, but also applies to any password that was typed correctly and then subsequently expired causing it to be invalid.
OS X Workarounds:

Open up keychain and kill the gimme-aws-creds entry using the UI
Use keyring set gimme-aws-creds<EMAIL_ADDRESS>from the terminal

Hmm it's not clear to me why this behavior started, but I just ran into it last week too.

Looks like a regression introduced in 29f5eef80e9dc3107083caf12364f37e26f5fa79 - the raise_for_status line here is preventing the expired password from being removed from the keyring on auth failure.

This has been fixed in v2.3.1, thanks!

This not only applies to incorrectly typed saved passwords, but also applies to any password that was typed correctly and then subsequently expired causing it to be invalid.
OS X Workarounds:

Open up keychain and kill the gimme-aws-creds entry using the UI
Use keyring set gimme-aws-creds<EMAIL_ADDRESS>from the terminal

How can we force gimme-aws-creds to enter password everytime without storing it in Keyring ? I dont have access to keyring and I assume I entered a incorrect password. I need to renter the correct password.
I am running this command gimme-aws-creds --profile my-profile --register_device

This is annoyingly happening in 2.4.3 after a password change. Any application using the keychain should fall back to a prompt and offer to update the keychain entry if the stored password is wrong.

This is annoyingly happening in 2.4.3 after a password change. Any application using the keychain should fall back to a prompt and offer to update the keychain entry if the stored password is wrong.

The first failure clears out the keychain, so you just have to run the command again

roboweaver -  nothing of the sort happens

@bschatz12 - Not sure what you mean by that - I have never had a problem with password reset other than remembering to run the command a second time in order to put the new password in place. Used to have to go into the keychain and manually find the password entry before they fixed that.

Same for sure - I haven't seen any cases where the keychain doesn't get cleared out.


Polymorphic store

Hi there,
I was wondering how to deal with polymorphic store.
I have two stores ActionStore and FollowUpStore and each one can receive comments.
var actionSchema = new Schema('actions', {idAttribute: 'id'});
var followUpSchema = new Schema('followUps', {idAttibute: 'id'});
var commentSchema = new Schema('comments', {idAttribute: 'id'});

actionSchema.define({
  comments: arrayOf(CommentSchema)
});

followUpSchema.define({
  comments: arrayOf(commentSchema)
});

comment.define({
  // followUp || action determine by the value of objectType field ('action' or 'followUp')
});

If I fetch by actions or followUps there is no problems. But when I store a new comment i need to push the new comment in the comments field in the ActionStore or the FollowUpStore.
Any idea ?
Aubin

+1

Sorry, I don't understand :-(
Can you provide more context?

Yes,
On my API I only use one table to store comments. A user can post comments for Actions or FollowUps.
To determine if a comment is link to an action or a followUps I use two fields : objectType (name of the model class), objectId (id of the action or followUp).
When I store a new comment I get this response:
{
  "id": 1,
  "comment": "a comment for the comment",
  "objectType": "action",
  "objectId": "1",
  "object": {
    "id": 1,
    "label": "a label",
    "comment": "a comment for the action"
  }
}

How to let normalizr parse my response to get the following result:
var normalize = {
  entities: {
    comments: [{
      id: 1,
      objectType: 'action',
      objectId: 1,
      comment: "a comment for the comment"
      action: 1
    }],
    // Only if objectType == 'action'
    actions: [{
      id: 1,
      label: "a label",
      comment: "a comment for the action"
      comments: [1]
    }]
  }
}

For now I've created two schemas for comment: ActionCommentSchema and FollowUpCommentSchema and I chose the one or the other depending on the context when the API respond.
I hope this is more explicit: I don't know if it is possible to solve this problem with normalizr.
Thanks in advance.

For now I've created two schemas for comment: ActionCommentSchema and FollowUpCommentSchema and I chose the one or the other depending on the context when the API respond.

This seems like the right solution to me. I'm closing, but if there is enough interest in this, I'd consider a PR implementing some sort of polymorphism support.


[Chore] Update Handlers to use new Meshkit Errors #2

Note: This is a child issue of #3275
Current Behavior
Meshery Server (and its other components, like Adapters) use a centralized library, MeshKit, to provide uniform utilities (e.g. raising and logging errors). Currently, Meshery Server uses a now antiquated style of error logging with logrus.
Desired Behavior
Ideally, all use of logrus is replaced with use of MeshKit-style error logging.
Implementation
Meshery Handlers needs to be:

Updated to use meshkit errors.
Switch to handler Log instead of logrus

Handlers to update:

[ ]  load test handler
[ ]  load test mesh handler
[ ]  load test preferences handler

For reference, you can checkout  the grafana handler and prometheus handler or the docs.
Define all Error functions for all the handlers to be written in the handlers/error.go file.
Acceptance Tests

Mockups

Contributor Guide

hey @debjani1413 if you haven't already finished, is there is any way i can work on this issue with you?

@warunicorn19 yes, sure let me assign it to :)

Thanks @debjani1413

@kumarabd this issue is not closed yet .


How to pass custom delimiters, dictionary and non-dictionary schemas

According to the paper, we can pass following configs for CLP.

delimiters
dictionary_variables
non_dictionary_variables

But, AFAIU, there is no way to pass these for clg and clp now.
Can you help me if I miss anything? Thanks

Hey @kavirajk,
Right, there’s currently no easy way to specify them. Full schema support is in our open-sourcing pipeline. We are finishing up optimization & testing. Stay tuned!

Thanks @kirkrodrigues for the information!.
So basically, now clp-core uses kinda static delimiter and variables check as done here and here respectively. Correct?
I also tested with your hadoop dataset. The compression ratio is very impressive!. Great work!. Looking forward for the full schema support!

Yup, that's correct.
Nice, thanks for trying it out! We'll post here when full schema support is open-sourced.

Hi @kirkrodrigues,
As mentioned in the paper, variable schema is pre-defined by the user. Are the regex as shown below currently manually written by the developer? I can see no part of the code checking for these schema. I understand it is not open-source yet, but can you please point out how is it done in currently?
dictionary_variables :
" task_ \ d + "                                              # Task ID
" \ d {1 ,3}\.\ d {1 ,3}\.\ d {1 ,3}\.\ d {1 ,3} " # IP
" container_ \ d + "                                     # Container ID

Hi @BasantaChaulagain,
Currently, we have a few schemas implemented implicitly (i.e., the logic is not implemented as a regular expression but as a set of conditions) in the code. Roughly, the logic works as follows:

EncodedVariableInterpreter::encode_and_add_to_dictionary repeatedly calls LogTypeDictionaryEntry::parse_next_var to parse every variable in a message.
LogTypeDictionaryEntry::parse_next_var calls get_bounds_of_next_var to parse the next variable in the message.
get_bounds_of_next_var iterates the message and gets the bounds of the next token in the message (a token starts with a non-delimiter (!is_delim) and ends before the next delimiter (is_delim).
get_bounds_of_next_var then runs a few checks to determine if the token should be treated as a variable according to the implicit schemas.
Once the control-flow returns to EncodedVariableInterpreter::encode_and_add_to_dictionary, it will try to convert the given variable into a non-dictionary variable (e.g., convert_string_to_representable_integer_var). If it cannot be converted, it will be treated as a dictionary variable.

Encoding a query is a little bit more complicated since we need to handle wildcards (admittedly, the logic is a bit messy). To see how it works, I would start from Grep::process_raw_query.
We are working hard to try and get an easy-to-use version of schema support open-sourced so that the logic won't be so complicated. I hoped we would be done by now, but development always takes longer than we expect.

Thank you for your response @kirkrodrigues. Good to hear that development is in progress.


ext_authz envoy filter Downstream protocol error

Bug description
When using the autogenerated cluster name for an external auth service, calls fail with a DPE(Downstream protocol error) flag. (Might be a duplicate of https://github.com/istio/istio/issues/21841).
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure
Expected behavior
Send working ext_authz /Check call without needing to patch the name of the cluster.
Steps to reproduce the bug
1: Deploy a service that implements the /Check endpoint according to envoy specs.
2: Check envoy configuration of istio-ingressgateway to see that a cluster configuration for the service has been added. (In my case called envoy-services and living in the envoy-services namespace)
2.1  istioctl dashboard envoy "ingress-gateway-pod-name"
2.2:
"cluster": {
    "@type": "type.googleapis.com/envoy.api.v2.Cluster",
    "name": "outbound|50051||envoy-services.envoy-services.svc.cluster.local",
    "type": "EDS",
    "eds_cluster_config": {
        "eds_config": {
             "ads": {}
          },
          "service_name": "outbound|50051||envoy-services.envoy-services.svc.cluster.local"
      },...

3:
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: ext-authz-filter
  namespace: istio-system
spec:
  workloadSelector:
    labels:
      istio: ingressgateway
  configPatches:
  - applyTo: HTTP_FILTER
    match:
      context: GATEWAY
      listener:
        portNumber: 443
        filterChain:
          filter:
            name: "envoy.http_connection_manager"
            subFilter:
              name: "envoy.router"
    patch:
      operation: INSERT_BEFORE
      value:
        name: envoy.ext_authz
        typed_config:
          "@type": type.googleapis.com/envoy.config.filter.http.ext_authz.v2.ExtAuthz
          failure_mode_allow: false
          grpc_service:
            envoy_grpc:
              cluster_name: "outbound|50051||envoy-services.envoy-services.svc.cluster.local"

Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
istioctl version --remote
client version: 1.6.5
control plane version: 1.6.5
data plane version: 1.6.5 (13 proxies)

kubectl version
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.0", GitCommit:"2bd9643cee5b3b3a5ecbd3af49d09018f0773c77", GitTreeState:"clean", BuildDate:"2019-09-19T13:57:45Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"16+", GitVersion:"v1.16.11-gke.5", GitCommit:"baccd25d44f1a0d06ad3190eb508784efbb990a5", GitTreeState:"clean", BuildDate:"2020-06-25T22:55:26Z", GoVersion:"go1.13.9b4", Compiler:"gc", Platform:"linux/amd64"}

How was Istio installed?
kubectl generated istio manifests as follows.
istioctl manifest generate -f istio-operator.yaml > generated.yaml
kubectl apply -f

Environment where bug was observed (cloud vendor, OS, etc)
GCP, Unix pods running in k8s.
Fixes
As mentioned in https://github.com/istio/istio/issues/21841 you can add a patch to the cluster naming, this will make the error go away.
This can be closed if its to much of a duplicate. Might be the error just surfaces in a different way in newer istio versions. But looking at it quickly the error seems to be something else.

Thanks for the report! I am nearly certain this is the same as https://github.com/istio/istio/issues/21841 since DPE is what would happen when we violate HTTP protocol (pipe in header), so lets merge. BTW, if you look in that issue we are looking into adding first class ext_authz support to help this. thanks!


Fragment Pager Adapter Injection + ViewModel + fragmentManager

Hello,
I have a FragmentAdapter with two constructor parameters: fragmentManager and a ViewModel.
I would like to inject it but I don't know what is the right way to put all the pieces together to declare the adapter in the koin module to be injected
The element that I need to inject:
class KeysPagerAdapter<TKey:BaseKey>(fragmentManager:FragmentManager, var viewModel:FocusableViewModel<TKey, List<TKey>>):       FragmentStatePagerAdapter(fragmentManager) {

where FocusableViewModel is an interface:
interface FocusableViewModel<TObject, TList:List<TObject>>: ObjectListViewModel<TObject, TList>

I want to create a keysPagerAdapter with the following viewModel injected:
class AnnotationsViewModel(contenderId: String, annotationId:String, val dao: ContenderDao): ViewModel(),
        FocusableViewModel<AnnotationDetailKey, List<AnnotationDetailKey>>

This viewModel is being created without problems to feed a fragment. It is defined in the koin module
viewModel {  params -> AnnotationsViewModel(params[EXTRA_CONTENDER_ID], params[EXTRA_ANNOTATION_ID], get()) }

and it is being used in a fragment:
val viewModel
            by viewModel<AnnotationsViewModel>{
                    mapOf(
                            Properties.EXTRA_CONTENDER_ID to (getKey<AnnotationsKey>()?.id?: Utils.EmptyUUID),
                            Properties.EXTRA_ANNOTATION_ID to Utils.EmptyUUID
                                    )
            }

So I need to be able to define the bean to inject the keysadapter and I'm not sure how to define it on the koin module and how to inject it to be used in a fragment.
I've tried with this:
 bean { params -> KeysPagerAdapter(params[FRAGMENT_MANAGER], get<AnnotationsViewModel>())}

And injecting it through:
val pagerAdapter by inject<KeysPagerAdapter<AnnotationDetailKey>>{ mapOf(Properties.FRAGMENT_MANAGER to childFragmentManager)}

But this fails at it complains about missing parameters EXTRA_CONTENDER_ID and EXTRA_ANNOTATION_ID.
It makes sense as it doesn't know how to resolve, but I don't know how to define the bean or how to pass the paramenters the inject to obtain it...

Hello,
If you want to use function parameters in your definition,
you just have to use the params arguments as you do. To inject parameters values, open a lambda behind by inject or by viewModel and fill a map of values.
But this is what you already done. I don't understand well where is the problem :/
Can you provide a small project to be able to better understand your case?


Remove outdated code

PULL REQUEST
Purpose and Content
Remove outdated code
Benefits and Risks
Remove outdated code
Linked Issues
(Provide references to any link issues. Use closes #issuenum to automatically close an open issue)

Fixes #309
Closes #309

PR Checklist

[x] This PR has a corresponding issue OR is linked to an SDI.
[x] I have followed CliMA's codebase contribution and style guidelines OR N/A.
[x] I have followed CliMA's documentation policy.
[x] I have checked all issues and PRs and I certify that this PR does not duplicate an open PR.
[x] I linted my code on my local machine prior to submission OR N/A.
[x] Unit tests are included OR N/A.
[x] Code used in an integration test OR N/A.
[x] All tests ran successfully on my local machine OR N/A.
[x] All classes, modules, and function contain docstrings OR N/A.
[x] Documentation has been added/updated OR N/A.

bors r+


how to use "xgboost cli_main.cc"

hi
how i should do if i use the "xgboost cli_main.cc"?
thanks

See  https://github.com/dmlc/xgboost/tree/master/demo/binary_classification


Question: Automatically commit objects for dedicated branches

Hi folks,
we are using lakefs to ingest automatically data via Kafka Connect to Minio. We are currently evaluating a procedure to automatically commit objects in lakefs branches. Are you providing some kind of configuration to auto commit for specific branches or do you have a best practice approach to do that via WebHooks respecitvely the lakefs API in general?
Thanks in advance
Domcek

Thanks @D0mc3k for bringing this up.
I can see the added value for ingestion pipelines, where you would like commits to happen automatically (like every X mins, every Y new objects?) since objects are always being uploaded.
There isn't a configuration for auto-committing of that sort to lakeFS. I don't think it should be too hard to support, @ozkatz wdyt? For now, you can use the commit API and call it every X mins by automating some logic on the client side.

depends on #2358


feat: support find by text and filter

This PR is related to #738
Goals:

[x] support find by text and filter for ElasticSearch
[x] also support that for Redis (if needed)
[x] add tests
[ ] update docs

Codecov Report
Base: 87.85% // Head: 75.77% // Decreases project coverage by -12.08% :warning:

Coverage data is based on head (44bae10) compared to base (bb9d1c3).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head 44bae10 differs from pull request most recent head ad6137d. Consider uploading reports for the commit ad6137d to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #740       +/-   ##
===========================================
- Coverage   87.85%   75.77%   -12.09%
===========================================
  Files         133      133
  Lines        6614     6613        -1
===========================================
- Hits         5811     5011      -800
- Misses        803     1602      +799

Flag
Coverage Δ

docarray
75.77% <100.00%> (-12.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

docarray/array/mixins/find.py
89.13% <100.00%> (-1.30%)
:arrow_down:

docarray/array/storage/elastic/find.py
96.36% <100.00%> (+0.06%)
:arrow_up:

docarray/math/distance/torch.py
0.00% <0.00%> (-100.00%)
:arrow_down:

docarray/math/distance/paddle.py
0.00% <0.00%> (-100.00%)
:arrow_down:

docarray/document/strawberry_type.py
0.00% <0.00%> (-100.00%)
:arrow_down:

docarray/math/distance/tensorflow.py
0.00% <0.00%> (-100.00%)
:arrow_down:

docarray/math/evaluation.py
0.00% <0.00%> (-94.83%)
:arrow_down:

docarray/array/mixins/embed.py
9.89% <0.00%> (-82.42%)
:arrow_down:

docarray/document/mixins/strawberry.py
16.27% <0.00%> (-79.07%)
:arrow_down:

docarray/array/mixins/evaluation.py
9.09% <0.00%> (-79.03%)
:arrow_down:

... and 35 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


VerifyContact component sends "true" instead of "email"

Do you want to request a feature or report a bug?
-A bug
What is the current behavior?
-Whit the VerifyContact component rendered and clicking in the radio button to verify email this function is executed in the AuthPiece component:
handleInputChange(evt) {
     this.inputs = this.inputs || {};
     const { name, value, type, checked } = evt.target;
     const check_type = ['radio', 'checkbox'].includes(type);
     this.inputs[name] = check_type? checked : value;
 }
since the type is "radio" the check_type will be assigned to true and the last row assigns "true" instead of "email" to this.inputs[name].
In the VerifyContact component when calling the verify function "true" is passed to Auth.verifyCurrentUserAttribute(contact)
What is the expected behavior?
Auth.verifyCurrentUserAttribute(contact) is called with the value "email"
Which versions of Amplify, and which browser / OS are affected by this issue? Did this work in previous versions?
tested in
"aws-amplify": "0.4.4" & "0.4.0",
"aws-amplify-react": "0.1.50" & "0.1.46",
did not work properly in any version
Chrome 66.0.3359.181
Windows 10

Also: Breaking the application in the debugger and changing the value from "true" to "email" makes it work.

PR #986 addresses this issue.


superagent 2.x deprecates setStatusProperties

Using response._setStatusProperties if available to avoid warning about superagent.2.x making setStatusProperties method private

Not sure why Travis didn't like previous PR. This is the same code change. Anyway, Travis is reporting ok now.

👍

:+1: Thanks for openning a second PR

:+1:

It seems there are some conflicts, could you fix them ? 😺

ok, updated PR. please merge before it goes stale again. Thanks

Thank you @awei01 😉

Tag v1.12.0


add reactive array swap

I am writing a simple todo list component with:
Vue.component('todo-list', {
    props: ['todos'],
    template:
    `

            {{td}}
            <button title="delete"  @click="todos.splice(ix,1)">X</button>
            <button title="move up" @click="todos.swap(ix,ix-1)" v-show="ix>0">^</button>

`,
})

For this to work I need (in core/observer/array.js) something like:
Array.prototype.swap = function(x,y) {
  var t = this[x]
  this[x] = this[y]
  this[y] = t
}
const arrayProto = Array.prototype
export const arrayMethods = Object.create(arrayProto)

/**
 * Intercept mutating methods and emit events
 */
;[
  'push',
  'pop',
  'shift',
  'unshift',
  'splice',
  'swap', // added
  'sort',
  'reverse'
]

It works when I (lazily) fixed dist/vue.js, but did not run the tests.
I think it is worth to add swap this way because notify() is run just once, am I wrong?

Hi @jcfg,
thanks for your suggested feature. So far, we have refrained from polluting Prototypes with custom methods - we had some in Vue 1.0 (Array.prototype.$set()) and explicitly removed them in 2.0.
I don't think  we should/ want to start adding that type of code to the framwork again unless the advantage it provides is extraordinary.
So far, I don't think that'S the case for your proposed .swap() method.

How about adding Vue.swap(), like Vue.set ?

Why not just write
Vue.swap = function(arr, x, y) {
   var origin = arr[x]
   arr.splice(x, 1, arr[y])
   Vue.set(arr, y, origin)
}

https://jsfiddle.net/L92w2372/2/

You are doing dependency notification "twice", 1st at splice, 2nd at Vue.set, am I right?
With adding Array.prototype.swap like above, or adding Vue.swap, we will do notifications just once.

You are doing dependency notification "twice", 1st at splice, 2nd at Vue.set, am I right?

Yes, but neither does that have any performance implications, nor is is calling any watcher's run() twice.
The update triggered by .notify() is pushed to the scheduler, which is working asynchronously and will skip any duplicate entries.

Ok, still I think it is worth to have a reactive swap in Vue. None of these are trivial to normal users.
Vue.swap also needs to do a single notify(), like the above corrected comment, since Vue developers know Vue better than normal users ;)

Ok, still I think it is worth to have a reactive swap in Vue. None of these are trivial to normal users.

I don't think that the three lines of code by @HerringtonDarkholme should be non-trivial to any dev who knows his JS enough to work with Vue successfully otherwise.
We try to keep the API surface of Vue lean. We could think of dozens of small convenvience functions like yours, but that would just inflate the API unnessessarily.

Quick question: notifications are for the whole todo list right? Single elements of the list do not have notifications of their own.
If they did, Vue.swap above would miss notifications on arr[x].
For example can we have something like:
data: {
msg: {x:1, y:2}, // reactive var
todos: [ msg, "td1", ..] // reactive list
}

We could think of dozens of small convenvience functions like yours, but that would just inflate the API unnessessarily.

Therefor one could aggregate helpers like this in a plugin and have the core clean.

Quick question: notifications are for the whole todo list right? Single elements of the list do not have notifications of their own.

In this case, they are., as we are changing an array.

Therefor one could aggregate helpers like this in a plugin and have the core clean.

Contreibution welcome.
I will close this issue now, since I think we have come to an understanding that this won't be added to core.


Provide sample_code_init_fields to samples via sample_value_sets; apply to NodeJS

For backward compatibility, sample_init_fields remains the source of
truth for IN_CODE samples for the moment. sample_init_fields is
now copied to the in-code sample view model so that as languages adopt
samplegen, those language's templates can stop depending on
method.initCode. Once all languages have done this mgration, we can
remove method.initCode

Do the copying from sample_init_fields in SampleTransformer
Make NodeJSPackageMetadataTransformer and
NodeJSGapicSurfaceTestTransformer use NodeJSMethodViewGenerator
Make the NodeJS in-code samples use the sample-configured init fields

One issue I have not figured out: how to create blank lines blank lines only between iterations of an @join in the templates. For spacing purposes, I had to work around my ignorance by creating a blank line at the end of each iteration, but that unfortunately leads to extra blank lines in the baselines (and doesn't look as pretty). If I can figure out how to do this properly, then the baseline files will not change in this PR.
I haven't spent much time researching how this might be doable. I vaguely recall it's being done somewhere else in our templates.  Otherwise, I will hunt for it next week.

Codecov Report

Merging #2009 into master will decrease coverage by <.01%.
The diff coverage is 94.54%.

@@             Coverage Diff             @@
##             master   #2009      +/-   ##
===========================================
- Coverage     85.81%   85.8%   -0.01%
- Complexity     5007    5010       +3
===========================================
  Files           457     457
  Lines         20106   20104       -2
  Branches       2109    2110       +1
===========================================
- Hits          17253   17251       -2
  Misses         2142    2142
  Partials        711     711

Impacted Files
Coverage Δ
Complexity Δ

...nsformer/nodejs/NodeJSGapicSurfaceTransformer.java
100% <ø> (ø)
27 <0> (ø)
:arrow_down:

...nsformer/nodejs/NodeJSGapicSamplesTransformer.java
100% <ø> (ø)
9 <0> (-1)
:arrow_down:

...ormer/nodejs/NodeJSPackageMetadataTransformer.java
90.35% <100%> (-0.41%)
21 <1> (ø)

...gle/api/codegen/transformer/SampleTransformer.java
98.24% <100%> (+0.24%)
13 <1> (+3)
:arrow_up:

...n/transformer/DynamicLangApiMethodTransformer.java
99.37% <100%> (ø)
32 <0> (ø)
:arrow_down:

...rmer/nodejs/NodeJSGapicSurfaceTestTransformer.java
100% <100%> (ø)
24 <1> (ø)
:arrow_down:

.../transformer/nodejs/NodeJSMethodViewGenerator.java
91.17% <90%> (+0.85%)
9 <7> (+1)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 265c052...3cc56a8. Read the comment docs.

Changes LGTM if those nasty empty lines are removed. Unfortunately I don't really know where it should be changed.

@alexander-fenster Should be fixed now (easier than I thought). PTAL.


Unexpected behavior for invariant checking

This example fails, but I was expecting it to go through.
import stainless.annotation._
import stainless.smartcontracts._
import Environment._

trait A extends Contract {
  @addressOfContract("A")
  var a: Address
  var x: BigInt

  final def constructor(_a: Address) = {
    a = _a
    dynAssert(contractAt(a).asInstanceOf[A].x >= 0)
  }

  final def invariant() = {
    contractAt(a).asInstanceOf[A].x >= 0
  }
}

Due to the annotation @addressOfContract("A"), you have to add to more assumption on a in the constructor : contractAt(a).isInstanceOf[A] && contractAt(a).isInstanceOf[A].addr == a. However, apparently this still fails, but it works if you do
dynAssert(contractAt(_a).isInstanceOf[A])
dynAssert(contractAt(_a).asInstanceOf[A].addr == _a)
dynAssert(contractAt(_a).asInstanceOf[A].x >= 0)
a = _a

Do you know why ?

Thanks that makes sense. There seems to be a bug (maybe from the way Stainless generates VCs) where the last dynAssert is not taken into account.

The bug was reported in https://github.com/epfl-lara/stainless/issues/531 and fixed in https://github.com/epfl-lara/inox/pull/99.


Make the copyright correct.

Travis failed

yeah.i saw

ive managed to reproduce this locally which is a good start

Superseded by https://github.com/gds-operations/puppet-auditd/pull/26.


No correct completion for df.loc

VSCode Version: 1.32.3
Extension Version: 2019.2.5558
OS: Windows 10
Python 3.7.2
LS Version: <IP_ADDRESS>
Using Conda virtual environment
Working with this example tutorial. Expected df.loc to give me a full list of options (df.loc[] was the option I was after), but I only recieved 2 as in the below figure (analysis had finished completing):

Please provide complete example with imports. Thanks.

import matplotlib.pyplot as plt
import numpy as np

# Plot Histogram on x

x = np.random.normal(size = 1000)
plt.hist(x, bins=50)
plt.gca().set(title = "Frequency Histogram", ylabel = 'Frequency')

# Pandas tutorial example script

import pandas as pd
df = pd.read_csv('https://raw.githubusercontent.com/selva86/datasets/master/diamonds.csv')

x1 = df.loc[df.cut=='Ideal', 'depth']
x2 = df.loc[df.cut == 'Fair', 'depth']
x3 = df.loc[df.cut=='Good', 'depth']

kwargs = dict(alpha = 0.5, bins = 100)

plt.hist(x1, **kwargs, color='g', label='Ideal')
plt.hist(x2, **kwargs, color='b', label='Fair')
plt.hist(x3, **kwargs, color='r', label='Good')
plt.gca().set(title='Probability Histogram of Diamond Depths', ylabel='Probability')
plt.xlim(50,75)
plt.legend()

@Anapo14 retry this.

Folding into #981


How does the MyFirstServer.bat for Windows work

Sorry in advance, I'm a noob. But with the most recent rocket for unturned it added two new folders and one new .bat file. The folders it added are a steam folder and an unturned folder, and they are both empty. Also it created a update.bat file in the scripts folder, and when i launch the myfirstrocketserver.bat, in the cmd it says "warning: unturned closed or crashed, restarting" then it spams the cmd with "unturned.exe is not recognized as an internal or external command, operable program or batch file". This is very frustrating and ive tried everything to my knowledge (which is pretty much nothing XD).
Edit: Also id like to note that i have been doing this private server thing for quite a while now and its always worked before on this computer so I just dont know anything about these new additions and it doesnt work like it used to. Try and be easy on me because i know nothing about coding or those sort of things and to me this is a struggle. XD

thank you for explaining the same problem i have perfectly xD

The new script works different.
How it works:

Create a new folder "MyFirstServer"
Paste the whole contents of the Rocket zip file
Go into the scripts folder, run MyFirstServer.bat, the script will then install SteamCMD and Unturned into /MyFirstServer/Unturned and /MyFirstServer/Steam, wait for it to finish, then it will start Unturned
Stop Unturned again, configure & install plugins, done
To update, download latest zip, copy over all the files in MyFirstServer and run update.bat

Sorry but once again didn't work, it did the exact same thing as before. It created a steam and unturned folder with nothing in it. After the MyFirstRocketServer.bat ran, a steamcmd.zip showed up in the scrips folder for a second then disappeared. I even tried to manually install steamcmd and that didn't work either.

Yes, thats the problem, you did exactly the same thing as before, and the script changed, please read my answer. And in addition it may be to say that you don't need those scripts, putting all the *.dll files into Unturned/Unturned_data/Managed/ works aswell.

Ok, I don't want to say your wrong or anything I just want to explain to you how i used to do it and how i tried it the new way with the new version (the way you explained it). So the old way i used to do it is i would copy the whole unturned game folder and paste it to my desktop, then i would rename the folder to "Unturned Rocket Server" I would then open the folder and go into unturned data, managed then i would pasted all of the contents from the rocket zip into the managed folder. Then i would go into scripts, run the myfirstrocketserver.bat and then shut it down once it is done loading, next i would go back to the unturned folder and go into servers, my first rocket server, change the server commands and add my plugins.
Now since this no longer works I came here to ask for help, and when I got your help i tried it many times EXACTLY how YOU said it, and i can't get it to work. I created a new folder, named it "My first server" extracted the rocket zip into the new empty folder (named "my first server"), I go into the scrips folder and run "myfirstrocketserver.bat", the console pops up like usual and it has a nice new little loading thing at the top in like a yellow green that says "reading web response" then changed to  "writing web request" and finishes loading. Then a red error pops up then shuts down the console, I am unable to read the error because it shuts down so fast, after it shuts down, in the scrips folder I see a steamcmd.zip file that disappears after one second. I back out of the scripts folder and I see a steam folder and a unturned folder, they are both empty.
Now i don't know if what I just explained is wrong and i just didn't understand your instructions, or if I'm doing it right and its just not working at all, and I know that first part where I explained how i used to make a server was probably unnecessary. Also i am using the newest rocket 809.

Well its possible there is an error in the script, i am not perfect ;D Do you have time for a Teamviewer session? Join the chat room on https://rocketmod.net/chat to coordinate a session...

About halfway through the reply i ran the myfirstserver.bat and by the time i sent it I checked it again. The steamcmd.zip stayed in the scripts folder, so I unzipped it and placed the steamcmd.exe in the empty steam folder and let it run, i now have a unturned folder with content. Im not sure how it worked, i did nothing different, maybe a bug? But i have to go to work right now and ill give you more details later and also let you know in a few hours if the server works. Thank you for the help.

Mhm.... could be a issue expanding the zip file (https://github.com/RocketMod/Rocket.Unturned/blob/master/Rocket.Unturned/Scripts/Windows/update.bat#L17)... Just found out its not available on Windows 7 / 8, i will look into finding another solution...

Ah ok, I am using Windows 8.1

No problem, i changed the script to use vbs to unpack, try latest version, either download Rocket or get it from here https://github.com/RocketMod/Rocket.Unturned/blob/master/Rocket.Unturned/Scripts/Windows/update.bat

Ok thank you I will try that when I get home

The new rocket version worked perfect, but doesn't the steamcmd.exe have to be moved into the steam folder and ran there? Because the steamcmd.exe is in the scrips folder

The script should move it to the Steam folder..


Allow the creation of pages that are not backed by a table

Is there any way to create a page that is not backed by a table?
I have a couple of pages that don't need any additional fields besides the ones provided by the Page model.

Hi @jorge-marques - please use the Wagtail Google group https://groups.google.com/forum/#!forum/wagtail for support queries.

Hi @gasman.
I've rephrased the issue, hopefully it's more adequate for github now.
I believe the example I provided makes some sense, at least conceptually. I'm trying to understand if this use case was anticipated by the wagtail developers. I believe this could be a useful feature/optimization.
If you still feel this is not the place for this discussion, please close the issue again and I'll try the support or maybe the developer group.

It's possible to create page types that have no additional fields - the HomePage class provided for a fresh project is an example: https://github.com/torchbox/wagtail/blob/master/wagtail/project_template/home/models.py
IIRC, this creates a table consisting only of a primary key, which isn't very pretty, but it does the job. Do you see any practical advantage to being able to define a model with no table at all?

Preventing Django from creating that table would be tricky. Things might get complicated if you decide to add a field to the empty model, which would mean that the table would also need to be created and records inserted for every existing instance of that page type. I imagine there would be many edge-cases that we would need to consider as well.
Perhaps you could use a proxy model instead? https://docs.djangoproject.com/en/1.8/topics/db/models/#proxy-models

@gasman I suppose it's not really a big deal. I have a website that's mostly static, but has a couple of managed page types. I found it easier to create wagtail pages for the "static" pages than to mix and match "manual" routes with wagtail managed pages. It would just be neater for my current use case, but the issue mentioned by @kaedroho is better avoided. It's no big deal.
@kaedroho, regarding the use of a proxy model, wagtail does not seem to pick it up, but maybe I'm doing something wrong:
class MyPage(Page):
    is_abstract = False

    class Meta:
        proxy = True
        verbose_name = 'My Page'


[Snyk] Upgrade axios from 1.1.3 to 1.2.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade axios from 1.1.3 to 1.2.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 3 versions ahead of your current version.
The recommended version was released a month ago, on 2022-12-05.

Release notes

Package name: axios

1.2.1 - 2022-12-05[1.2.1] - 2022-12-05
Changed

feat(exports): export mergeConfig #5151

Fixed

fix(CancelledError): include config #4922
fix(general): removing multiple/trailing/leading whitespace #5022
fix(headers): decompression for responses without Content-Length header #5306
fix(webWorker): exception to sending form data in web worker #5139

Refactors

refactor(types): AxiosProgressEvent.event type to any #5308
refactor(types): add missing types for static AxiosError.from method #4956

Chores

chore(docs): remove README link to non-existent upgrade guide #5307
chore(docs): typo in issue template name #5159

Contributors to this release

Dmitriy Mozgovoy
Zachary Lysobey
Kevin Ennis
Philipp Loose
secondl1ght
wenzheng
Ivan Barsukov
Arthur Fiorette

1.2.0 - 2022-11-22[1.2.0] - 2022-11-10
Changed

changed: refactored module exports #5162
change: re-added support for loading Axios with require('axios').default #5225

Fixed

fix: improve AxiosHeaders class #5224
fix: TypeScript type definitions for commonjs #5196
fix: type definition of use method on AxiosInterceptorManager to match the the README #5071
fix: __dirname is not defined in the sandbox #5269
fix: AxiosError.toJSON method to avoid circular references #5247
fix: Z_BUF_ERROR when content-encoding is set but the response body is empty #5250

Refactors

refactor: allowing adapters to be loaded by name #5277

Chores

chore: force CI restart #5243
chore: update ECOSYSTEM.md #5077
chore: update get/index.html #5116
chore: update Sandbox UI/UX #5205
chore:(actions): remove git credentials after checkout #5235
chore(actions): bump actions/dependency-review-action from 2 to 3 #5266
chore(packages): bump loader-utils from 1.4.1 to 1.4.2 #5295
chore(packages): bump engine.io from 6.2.0 to 6.2.1 #5294
chore(packages): bump socket.io-parser from 4.0.4 to 4.0.5 #5241
chore(packages): bump loader-utils from 1.4.0 to 1.4.1 #5245
chore(docs): update Resources links in README #5119
chore(docs): update the link for JSON url #5265
chore(docs): fix broken links #5218
chore(docs): update and rename UPGRADE_GUIDE.md to MIGRATION_GUIDE.md #5170
chore(docs): typo fix line #856 and #920 #5194
chore(docs): typo fix #800 #5193
chore(docs): fix typos #5184
chore(docs): fix punctuation in README.md #5197
chore(docs): update readme in the Handling Errors section - issue reference #5260 #5261
chore: remove \b from filename #5207
chore(docs): update CHANGELOG.md #5137
chore: add sideEffects false to package.json #5025

Contributors to this release

Maddy Miller
Amit Saini
ecyrbe
Ikko Ashimine
Geeth Gunnampalli
Shreem Asati
Frieder Bluemle
윤세영
Claudio Busatto
Remco Haszing
Dmitriy Mozgovoy
Csaba Maulis
MoPaMo
Daniel Fjeldstad
Adrien Brunet
Frazer Smith
HaiTao
AZM
relbns

1.2.0-alpha.1 - 2022-11-10[1.2.0-alpha.1] - 2022-11-10
Changed

changed: refactored module exports #5162
change: re-added support for loading Axios with require('axios').default #5225

Fixed

fix: improve AxiosHeaders class #5224
fix: TypeScript type definitions for commonjs #5196
fix: type definition of use method on AxiosInterceptorManager to match the the README #5071
fix: __dirname is not defined in the sandbox #5269
fix: AxiosError.toJSON method to avoid circular references #5247
fix: Z_BUF_ERROR when content-encoding is set but the response body is empty #5250

Refactors

refactor: allowing adapters to be loaded by name #5277

Chores

chore: force CI restart #5243
chore: update ECOSYSTEM.md #5077
chore: update get/index.html #5116
chore: update Sandbox UI/UX #5205
chore:(actions): remove git credentials after checkout #5235
chore(actions): bump actions/dependency-review-action from 2 to 3 #5266
chore(packages): bump loader-utils from 1.4.1 to 1.4.2 #5295
chore(packages): bump engine.io from 6.2.0 to 6.2.1 #5294
chore(packages): bump socket.io-parser from 4.0.4 to 4.0.5 #5241
chore(packages): bump loader-utils from 1.4.0 to 1.4.1 #5245
chore(docs): update Resources links in README #5119
chore(docs): update the link for JSON URL #5265
chore(docs): fix broken links #5218
chore(docs): update and rename UPGRADE_GUIDE.md to MIGRATION_GUIDE.md #5170
chore(docs): typo fix line #856 and #920 #5194
chore(docs): typo fix #800 #5193
chore(docs): fix typos #5184
chore(docs): fix punctuation in README.md #5197
chore(docs): update readme in the Handling Errors section - issue reference #5260 #5261
chore: remove \b from filename #5207
chore(docs): update CHANGELOG.md #5137
chore: add sideEffects false to package.json #5025

Contributors to this release

Maddy Miller
Amit Saini
ecyrbe
Ikko Ashimine
Geeth Gunnampalli
Shreem Asati
Frieder Bluemle
윤세영
Claudio Busatto
Remco Haszing
Dmitriy Mozgovoy
Csaba Maulis
MoPaMo
Daniel Fjeldstad
Adrien Brunet
Frazer Smith
HaiTao
AZM
relbns

1.1.3 - 2022-10-15Added
Added custom params serializer support #5113
Fixed
Fixed top-level export to keep them in-line with static properties #5109
Stopped including null values to query string. #5108
Restored proxy config backwards compatibility with 0.x #5097
Added back AxiosHeaders in AxiosHeaderValue #5103
Pin CDN install instructions to a specific version #5060
Handling of array values fixed for AxiosHeaders #5085
Chores
docs: match badge style, add link to them #5046
chore: fixing comments typo #5054
chore: update issue template #5061
chore: added progress capturing section to the docs; #5084
Contributors to this release

Jason Saayman
scarf
Lenz Weber-Tronic
Arvindh
Félix Legrelle
Patrick Petrovic
Dmitriy Mozgovoy
littledian
ChronosMasterOfAllTime
Salman Shaikh

    from axios GitHub release notes

Commit messages

Package name: axios

1fd79d3 chore: release 1.2.1
a6efeaf Change AxiosProgressEvent.event type to `any`; (#5308)
6d367b9 docs: remove README link to non-existent upgrade guide (#5307)
2687460 feat: export mergeConfig (#5151)
6f59cdb refactor: add missing types for static AxiosError.from method (#4956)
e1a4efe Fix typo in issue template name (#5159)
e3d7594 fix: exception to sending formdata in webworker (#5139)
9041c7d Fixed decompression for responses without `Content-Length` header (#5306)
786b113 fix: removing multiple/trailing/leading whitespaces (#5022)
cac6cfa fix: include config on new CancelledError (#4922)
f92e167 chore(release): 1.2.0
bc09c90 chore(docs): updated changelog
d032edd Refactored adapters loader; (#5277)
a3d9017 Fixed Z_BUF_ERROR when content-encoding is set but the response body is empty; (#5250)
b7ee49f Added toJSONObject util; (#5247)
a372b4c update readme (#5261)
e05fe7c Update the link (#5265)
7a4c0ea __dirname is not defined (#5269)
86eeff0 ci: remove git credentials after checkout (#5235)
2c83d47 chore(deps): bump loader-utils from 1.4.1 to 1.4.2 (#5295)
f79cf7b chore(deps): bump actions/dependency-review-action from 2 to 3 (#5266)
e1989e9 chore(deps): bump engine.io from 6.2.0 to 6.2.1 (#5294)
3a7c363 chore: v1.2.0 pre-release
bb59a73 chore: release v1.2.0

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

oppdatert i tidligere release


Possibly wrong bounds for lsp-test

This pops up when using cabal new-build, see https://github.com/bubba/lsp-test/commit/91086dedbd6ee34ce5c9f3cc47081d839882e29c#commitcomment-33279281.

Has been fixed with a revision on hackage

I'm still observing this, as is @power-fungus, even after updating my hackage index. Possibly best to just push a restriction to the bounds in this repo for now.

There was some issue that has been solved by a hackage revision. However the issue about incorrect bounds is still present (see #1210, sorry for the duplicate issue) a quick PR #1211 fixes this issue temporarily.

Sometimes you have to tell stack to use a specific revision, for some reason. e.g. see monad-dijkstra in the current deps.

The problem is with cabal new-build, not stack, stack seems to work just fine.

For stack we actually fix the version to the correct one.


When using multicursor, Home and End keys do not scroll.

VSCode Version: 1.5.3
OS Version: Windows 8.1

Steps to Reproduce:

Make some long texts until appears the horizontal scroll on the editor.
Use multicursor in some or all of these lines.
Press Home or End key to see what happens.

In this case (where the multiple cursors are close, we could scroll), but in the general case (where you could have multiple cursors on hundreds of distinct lines that do not fit within the viewport) this is impossible to do.

@alexandrudima a good example you could take is the Sublime Text editor, it has multiple cursors support and this issue is not present here 😄.

@sant123 Thanks. Have you tried adding 1000 cursors on 1000 different rows?

I haven't had the need to add 1000 cursors @alexandrudima 😃. However, take this as an example:
I have 75 lines in a file, but I only see 48 because of my monitor. Now I add the multicursor on the 75 lines of my file, then I press home and end keys. What should it do? Move the cursors to the beginning or the end of each row. I don't have to see the others because I know the others are in the beginning or in the end.
Is this what you are trying to tell me?

Duplicate of #6995


Update Helm repo location on Bintray

We still have a number of references to deis/helm on bintray:

https://bintray.com/deis/helm-ci/helm
https://bintray.com/deis/helm/helm

@sgoings if you want to add me to the bintray account, I'm https://bintray.com/slack

https://bintray.com/helm/helm and https://bintray.com/helm/helm-ci are now configured.
Todos:

[ ] Publish all released bins from deis/helm to helm/helm
[ ] Update CI & bintray deploy logic
[ ] Update micro-kube integration
[ ] Update helm.sh installer


Cannot add breakpoints on php files anymore

VSCode Version:1.52.0
OS Version:Linux x64 5.4.0-58-generic

Steps to Reproduce:
1.Cannot add any new breakpoint on php files
Existing breakpoints before the update still remain, can add breakpoint in other files in the same project, not php, can be added.
Other non php projects can add breakpoints normally.

Does this issue occur when all extensions are disabled?: Yes

This has been marked as fixed in 1.52.1, but i still cant place breakpoints.

/duplicate #112288
just update

/duplicate #112288


Null check operator used on a null value when running build_runner

I've decided to update Moor to latest version and now I'm getting following crash when trying to run flutter packages pub run build_runner build --delete-conflicting-outputs -v
Any help appreciated as I have no idea what's wrong.
...
[SEVERE] moor_generator:moor_generator on lib/about.dart:

Null check operator used on a null value
#0      LinkedElementFactory.isLibraryUri (package:analyzer/src/summary2/linked_element_factory.dart:418:48)
#1      LibraryContext.isLibraryUri (package:analyzer/src/dart/analysis/library_context.dart:95:27)
#2      LibraryAnalyzer._isLibrarySource (package:analyzer/src/dart/analysis/library_analyzer.dart:511:25)
#3      LibraryAnalyzer._resolveDirectives (package:analyzer/src/dart/analysis/library_analyzer.dart:552:36)
#4      LibraryAnalyzer.analyzeSync (package:analyzer/src/dart/analysis/library_analyzer.dart:128:5)
#5      LibraryAnalyzer.analyze (package:analyzer/src/dart/analysis/library_analyzer.dart:105:12)
#6      AnalysisDriver._computeAnalysisResult.<anonymous closure> (package:analyzer/src/dart/analysis/driver.dart:1590:63)
#7      PerformanceLog.run (package:analyzer/src/dart/analysis/performance_logger.dart:32:15)
#8      AnalysisDriver._computeAnalysisResult (package:analyzer/src/dart/analysis/driver.dart:1565:20)
#9      AnalysisDriver._computeErrors (package:analyzer/src/dart/analysis/driver.dart:1644:26)
#10     AnalysisDriver.performWork (package:analyzer/src/dart/analysis/driver.dart:1266:20)
#11     AnalysisDriverScheduler._run (package:analyzer/src/dart/analysis/driver.dart:2296:24)
<asynchronous suspension>

package:analyzer/src/dart/analysis/driver.dart 1619:9             AnalysisDriver._computeAnalysisResult.<fn>
package:analyzer/src/dart/analysis/performance_logger.dart 32:15  PerformanceLog.run
package:analyzer/src/dart/analysis/driver.dart 1565:20            AnalysisDriver._computeAnalysisResult
package:analyzer/src/dart/analysis/driver.dart 1644:26            AnalysisDriver._computeErrors
package:analyzer/src/dart/analysis/driver.dart 1266:20            AnalysisDriver.performWork
package:analyzer/src/dart/analysis/driver.dart 2296:24            AnalysisDriverScheduler._run

[INFO] Heartbeat:18.7s elapsed, 31/47 actions completed.
[INFO] Heartbeat:19.7s elapsed, 31/47 actions completed.
[INFO] Heartbeat:20.7s elapsed, 31/47 actions completed.
[INFO] Heartbeat:21.8s elapsed, 31/47 actions completed.
[INFO] Heartbeat:22.9s elapsed, 31/47 actions completed.
[INFO] Heartbeat:23.9s elapsed, 31/47 actions completed.
[INFO] Heartbeat:25.0s elapsed, 31/47 actions completed.
[INFO] Heartbeat:26.0s elapsed, 31/47 actions completed.
[INFO] Heartbeat:27.1s elapsed, 31/47 actions completed.
[INFO] Heartbeat:28.1s elapsed, 31/47 actions completed.
[INFO] Heartbeat:29.1s elapsed, 31/47 actions completed.

flutter doctor
[√] Flutter (Channel stable, 2.0.5, on Microsoft Windows [Version 10.0.19042.928], locale pl-PL)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[X] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[√] Android Studio (version 4.1.0)
[√] VS Code (version 1.55.2)
[√] Connected device (2 available)

pubsec.yaml
  cupertino_icons: ^0.1.3
  flutter_bloc: ^4.0.0
  http: ^0.13.0
  shimmer: ^1.1.1
  moor_flutter: ^4.0.0
  moor: ^4.2.1
  provider: ^4.2.0
  cached_network_image: ^3.0.0
  flutter_inapp_purchase: ^4.0.2
  rate_my_app: ^0.6.1+6
  simple_auth: ^2.0.7
  simple_auth_flutter: ^2.0.7
  dio: ^4.0.0
  sprintf: "^4.0.0"
  url_launcher: ^6.0.3
  firebase_core: ^1.0.3
  firebase_crashlytics: ^2.0.0
  firebase_messaging: ^9.1.1
  package_info: ^0.4.3
  modal_bottom_sheet: ^2.0.0
  another_flushbar: ^1.10.17
  flutter_svg: ^0.22.0
  webview_flutter: ^1.0.7
  delayed_display: ^1.1.0
  lottie: ^1.0.1
  launch_review: ^3.0.0
  smooth_page_indicator: ^0.2.0
  auto_size_text: ^2.1.0
  flutter_markdown: ^0.6.1
  google_nav_bar: ^5.0.3
  preload_page_view: ^0.1.6
  image_picker: ^0.7.4

dev_dependencies:
  flutter_test:
    sdk: flutter
  moor_generator: ^4.2.2
  build_runner: ^2.0.1

I'm guessing that this might be the solution: https://github.com/dart-lang/sdk/pull/45823

True, that does look similar. I'll close this issue since it looks like it's not moor's fault :D


⚠ Bofrost.fr has degraded performance

In 2755378, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 942 ms

Resolved: Bofrost.fr performance has improved in 07af819.


[opencv] enabling cublas

If I don't enable cublas several cuda headers do not get installed such as, cudaarithm.hpp, cudabgseg.hpp, cudacodec.hpp, cudafeatures2d.hpp.
When trying to use cv::cuda::ORB I need the header cudafeatures2d.hpp but the header was not found. After enabling cublas the headers were installed.
I tested on Windows 10 with visual studio 2017 and it works.
Thanks,
Ahmad

I don't think cublas is currently supported on x86.   But, the x86 opencv[cuda] port doesn't work either, so maybe it doesn't matter.

I tested it on x64 and it works. If cuda is enabled why not cublas?

Did you test on x86?

I did not.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ahmadhasan2k8 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @ahmadhasan2k8,
Thanks for your contribution!
This PR passes the CI tests, so I will go ahead and merge it.

Thank you.

Thanks


Better version sorting and defaults

I intended for this to only be fix for #19, but I noticed that the default version (if none is specified) was the earliest instead of the latest.
Fixes #19
Fixes #32

Looks good. Thanks!


[$250] Thread – Group chat opens when leave thread and reopen it

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.21-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4865736
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Log in
Navigate to any group chat
Send a message in the group chat
Open context menu and click on Reply in thread
Send a comment in the thread
Click on the header and Leave the thread
Verify you are redirected to group chat
Click on Reply to reopen the thread

Expected Result:
Thread opens
Actual Result:
Thread appears for a moment and redirect to group chat
Workaround:
Unknown
Platforms:

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/f9a3af3c-7c8c-4f4d-8c3b-0fe05bf3aa9a

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ce5b7018d6f6dd8e
Upwork Job ID:<PHONE_NUMBER>602372137
Last Price Increase: 2024-08-19

We think that this bug might be related to #vip-vsb

@jliexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Still looks to be happening on v9.0.21-3

Proposal
Please re-state the problem that we are trying to solve in this issue.
Thread appears for a moment and redirect to group chat
What is the root cause of that problem?
After we leave a group chat thread and open it again, GetMissingOnyxMessages is called and returns the data that sets the thread report as null (see the image below).

Then because the report is empty and this thread is also a group chat,  isRemovalExpectedForReportType is true and we will navigate back to parent report here
https://github.com/Expensify/App/blob/62722e8bc636f1be2e2c9e8e980aac93b76804c9/src/pages/home/ReportScreen.tsx#L581-L583
What changes do you think we should make in order to solve the problem?
We should use isRootGroupChat check here that also covers the isDeprecatedGroupDM case and exclude the case the thread is a group chat
const isRemovalExpectedForReportType =
     isEmpty(report) &&
     (ReportUtils.isMoneyRequest(prevReport) || ReportUtils.isMoneyRequestReport(prevReport) || ReportUtils.isPolicyExpenseChat(prevReport) || ReportUtils.isRootGroupChat(prevReport));

https://github.com/Expensify/App/blob/62722e8bc636f1be2e2c9e8e980aac93b76804c9/src/pages/home/ReportScreen.tsx#L581-L583
https://github.com/Expensify/App/blob/62722e8bc636f1be2e2c9e8e980aac93b76804c9/src/libs/ReportUtils.ts#L6944-L6946
What alternative solutions did you explore? (Optional)
NA

Proposal
Please re-state the problem that we are trying to solve in this issue.
Opening a group chat occurs when leaving and reopening a thread.
What is the root cause of that problem?
There is inconsistencies between FE and BE, while FE only hide the notification preferences. But BE set the report to null.
What changes do you think we should make in order to solve the problem?
If we want to handle it in BE then BE should sent the below code instead of setting report to null.
{
                          notificationPreference: CONST.REPORT.NOTIFICATION_PREFERENCE.HIDDEN,
                          participants: {
                              [currentUserAccountID]: {
                                  hidden: true,
                              },
                          },
                      }

if we want to resolve it in FE, we can remove || isChatThread [here] (https://github.com/Expensify/App/blob/62722e8bc636f1be2e2c9e8e980aac93b76804c9/src/libs/actions/Report.ts#L2811) and here.
What alternative solutions did you explore? (Optional)
N/A

The backend fix took care of this, closing this out.


change the recent activity log for retracted registration

Purpose
Change the recent activities field for retracted registration to be a warning instead of showing logs. Closes [#OSF-3969]
Changes
before:

after:

CR1 passed. Everything looks fine to me :+1:

Also- I've never actually used this feature before. Can you please flowdock me with notes on how to access this feature, so that I can preview the expected behavior in my browser?
(eg what page I go to in order to see a list of activity on that registration, and how to anonymize users in a registration)

Pass 2 finished. Nothing major, but two small notes to be addressed, and I'd like to run this once in the browser to confirm behavior before passing QA.

Pass 2 return complete. Looks good to me. :cactus:
I've noted one design question on JIRA that's outside the scope for this PR.


There is no desktop icon on the project

There needs to be a desktop icon of this project. As of now there is none on linux. I am not sure whether this is linux only or this problem also exists on other platforms

I also do not have an icon on Kubuntu 19.04 (KDE)

@SarathSantoshDamaraju this is linux only issue right?

Replicated in Debian Buster 10.1 XFCE.

Linux Mint 19.3 no icon as well


GO plugin /usr/local/share/lua/5.1/kong/init.lua:490: is enabled but not installed

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
Kong 2.7
Current Behavior
nginx: [error] init_by_lua error: /usr/local/share/lua/5.1/kong/init.lua:541: error loading plugin schemas: on plugin 'hello': hello plugin is enabled but not installed;
no plugin found
stack traceback:
	[C]: in function 'assert'
	/usr/local/share/lua/5.1/kong/init.lua:541: in function 'init'
	init_by_lua:3: in main chunk
Error: /usr/local/share/lua/5.1/kong/cmd/start.lua:64: failed to start nginx (exit code: 1)

Expected Behavior
My Docker file:
FROM golang:1.17.5-alpine3.14 AS builder

RUN mkdir -p /tmp/go/src/
COPY . /tmp/go/src

RUN cd /tmp/go/src/hello && go build -o hello
RUN ls /tmp/go/src/hello

FROM kong:2.7.0-alpine
USER root
COPY --from=builder  /tmp/go/src/hello/hello /usr/local/bin/
COPY --from=builder  /tmp/go/src/kong.conf /etc/kong

My GO File
package main

import (
	"fmt"
	"log"

	"github.com/Kong/go-pdk"
	"github.com/Kong/go-pdk/server"
)

func main() {
	server.StartServer(New, Version, Priority)
}

var Version = "0.2"
var Priority = 1

type Config struct {
	Message string
}

func New() interface{} {
	return &Config{}
}

func (conf Config) Access(kong *pdk.PDK) {
	host, err := kong.Request.GetHeader("host")
	if err != nil {
		log.Printf("Error reading 'host' header: %s", err.Error())
	}

	message := conf.Message
	if message == "" {
		message = "hello"
	}
	kong.Response.SetHeader("x-hello-from-go", fmt.Sprintf("Go says %s to %s", message, host))
}

kong.conf
log_level = debug
database = off
plugins = bundled,hello
#server
pluginserver_names = hello
pluginserver_hello_socket = /usr/local/kong/hello .socket
pluginserver_hello_start_cmd = /usr/local/bin/hello
pluginserver_hello_query_cmd = /usr/local/bin/hello -dump

port_maps=80:8000, 443:8443
proxy_listen = <IP_ADDRESS>:8000, <IP_ADDRESS>:8443 ssl http2
admin_listen = <IP_ADDRESS>:8001,<IP_ADDRESS>:8444 ssl
status_listen = <IP_ADDRESS>:8100

Steps To Reproduce
docker build -t gw-kong:1.0 .
docker run -it  7b382c5a180d /bin/bash
kong start -c /etc/kong/kong.conf

Anything else?
No response

I know where the reason is .!!

how was this resolved ? Any suggestions? I have same issue with Kong2.8

I resolved this by changing owner of go binary. You may change Dockerfile as:
COPY --from=builder --chown=kong /tmp/go/src/hello/hello /usr/local/bin/


kafkacat: 1.6.0 -> 1.7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/edenhill/kafkacat/releases.
meta.description for kafkacat is: "A generic non-JVM producer and consumer for Apache Kafka"
meta.homepage for kafkacat is: "https://github.com/edenhill/kafkacat"
meta.changelog for kafkacat is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.7.0 in filename of file in /nix/store/5c5800ysh9m1594c0a3mm8jd034qr6xf-kafkacat-1.7.0

directory tree listing: https://gist.github.com/292df673c49fddcb5c5a609e7d29be52

du listing: https://gist.github.com/be06588cd67c03ff6fee6d1e896e0f25

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
kafkacat

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5c5800ysh9m1594c0a3mm8jd034qr6xf-kafkacat-1.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A kafkacat https://github.com/r-ryantm/nixpkgs/archive/65bbcd78d992f5c0c7276b326885051f1758203e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5c5800ysh9m1594c0a3mm8jd034qr6xf-kafkacat-1.7.0
ls -la /nix/store/5c5800ysh9m1594c0a3mm8jd034qr6xf-kafkacat-1.7.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

kafkacat

Maintainer pings
cc @nyarly for testing.

Superseded by PR #140829


Searchable doesn't work with withCount?

The search method works fine when there's no withCount:
            $workers = Worker::search($term)
                ->with('photo')
                ->where('main_photoid', '!=', 0)
                ->where('status', 'enabled')
                ->paginate(24);

But if there is withCount - then it doesn't work - 0 results:
            $workers = Worker::search($term)
                ->withCount('comments')
                ->withCount('users')
                ->with('photo')
                ->where('main_photoid', '!=', 0)
                ->where('status', 'enabled')
                ->paginate(24);

I've also tried to put search($term) after withCount and it's the same:
            $workers = Worker::withCount('comments')
                ->withCount('users')
                ->search($term)
                ->with('photo')
                ->where('main_photoid', '!=', 0)
                ->where('status', 'enabled')
                ->paginate(24);

I am using Laravel 5.2.

Hey, PR most welcome, however withCount adds joins to the query, so it might be tricky to marry the two together ;)


Admin panel

Summary
This PR introduces a BackOffice for the project to help us with the management of the different resources running in the cluster.

The back office is configured in: /backoffice end-point.
Some logic was added be able to search by different resources and some fields that can contain some IP are hidden.
I refactorized a little bit the URLs file with the purpose to introduce STATIC folder in production and load the different css.
To access to the admin panel we will need to create some admin users so we can just create them through our environment variables in our helm. I will take a look into this.

Btw, the good news is that as soon as this is merge it we will be able to remove the tasks that we created to fill data 🎉

I'm not familiar this this admin module.  It looks useful.  Is it safe to enable this?  How is the user activated and the credentials are saved?

I need to admit that I'm not an expert either so I was basically reading a lot of documentation and asking:
https://docs.djangoproject.com/en/4.2/ref/contrib/admin/

Is it safe to enable this?

I would say yes. It would be as safe as any UI that requires authentication that we could have.

How is the user activated and the credentials are saved?

This is the process that I commented to you Aki, I don't know if you remember. Basically we can create an admin user with our current environment variables configuration: https://github.com/Qiskit/qiskit-serverless/blob/main/charts/qiskit-serverless/values.yaml#L93
This user is what we can use to access to the admin panel. I need to check how to create a secret in our Secrets-Manager for the initial password to use it in the environment variable. And with that we can create users for each of us.
Those admin users are saved the same as we are saving our current users. The only difference is that those users will have a password that django encrypts in database without problem.

The secret manger is not installed in the cluster.  For now, we need to get console access (kubectl) and create the secret using it.

As far as it safe.  I approve this PR.

As far as it safe. I approve this PR.

Thanks @akihikokuroda , I will wait for @IceKhan13 in case he would have any recommendation too 👍

The secret manger is not installed in the cluster.

I confirm that we have connected the secrets-manager to our clusters, Aki. I learned today how it works in the new account, I can ping you tomorrow and show it to you.

Merging by now to move forward. Feel free to check it whenever you want @IceKhan13 👍


HTTP Task of type POST doesn't get body from request

Hi,
I created a simple HTTP Task of type POST which passes the workflow's input as the body param.
When I trigger the workflow, Conductor does call the endpoint, but it doesn't pass the body.
I suspect that this is a bug, but if not I would appreciate your help understanding what am I missing.
In the conductor UI, I can see that the Task input looks good:
{
  "http_request": {
    "uri": "http://localhost:4800/date-to-year",
    "method": "POST",
    "contentType": "application/json",
    "body": {
      "date": "1800-01-01"
    }
  }
}

workflow definition:
{
  "name": "liran-date-to-year-rest-post",
  "description": "Gets date and returns year from it",
  "tasks": [
	{
      "name": "liran-task",
      "taskReferenceName": "liran-task",
      "inputParameters": {
		  "http_request": {
		    "uri": "http://localhost:4800/date-to-year",
		    "method": "POST",
    		"contentType": "application/json",
    		"body": "${workflow.input}"
		  }
      },
      "type": "HTTP"
	}
  ],
  "outputParameters": {
    "finalYear": "${liran-task.output.response.body.year}"
  },
  "schemaVersion": 2
}

execution example:
curl -X POST
http://localhost:8080/api/workflow/liran-date-to-year-rest-post
-H 'Content-Type: application/json'
-d '{"date": "1800-01-01"}'
I also tried to define the workflow with constant body
"inputParameters": {
  "http_request": {
    "uri": "http://localhost:4800/date-to-year",
    "method": "POST",
	"contentType": "application/json",
	"body": {"date": "2000-01-01"}
  }

just for the test to see if my endpoint gets the body.. but still it didn't
thank you!

Can you let me know where did you configure  your workflow definition, so that the conductor server will instantiate?

Sorry, I didn't understand your question.. What do you mean by "where"?
On Sat, Aug 4, 2018, 20:01 dileepkeely<EMAIL_ADDRESS>wrote:

Can you let me know where did you configure your workflow definition, so
that the conductor server will instantiate?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Netflix/conductor/issues/673#issuecomment-410463174,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACx0Kg8kGBBJ5T-3ca-D9KvqeSa7WzN8ks5uNdOCgaJpZM4Vaesj
.

--
Sent from my phone

I think I am facing this issue as well, any help is much appreciated.

I am facing same issue in v2.25.9, any suggestion would be great.


Scheduling api url is not correctly composed.

Issue Summary
When I debugged your scheduling process, I noticed that api requests from post-scheduling adapter goes with double-slash part in url (see screenshot https://cl.ly/2K2a1P2N1Q0t).
That's happened cause here https://cl.ly/352p1K1f3Z3Z in your code, apiUrl is already with trailing slash.
I had a problem with that, cause my NGINX configured to remove double(tripple ..) slashes, and your scheduling doesn't works properly.
Thanks

Hey @sudosu! Thanks for reporting this. I can confirm this bug and will work on a fix for it.

@AileenCGN The double slashes should exist on LTS as well.

I didn't see them. But this might we due to a missing trailing slash in the config. Will push a fix for LTS as well.


i want add React CSS Modules

hi
i want add React CSS Modules
https://github.com/gajus/react-css-modules
please help me
add css-loader style-loader webpack

I am not able to make it work... And hasn't anybody yet tried this. I mean css-modules


Convenient Option flatmap null handling

I understand the basis for requiring a flatMap to transform a potentially null-returning function on an Option value, but the current API (0.10.2 and 1.0.0-a3) is exceedingly clumsy to use, when in most realistic cases a Function<T, @Nullable U> is available. This leads to boilerplate like this:
private static <T, U> Function<? super T, Option<? extends U>> maybeNull(Function<? super T, ? extends U> wrapped) {
    return (input) -> Option.of(wrapped.apply(input));
}

It would be significantly convenient to have a method called something like flatMapUsing, analogous to mapTry, that implemented the Optional#map behavior:
default <U> Option<U> flatMapUsing(Function mapper) {
  return isEmpty() ? none() : Option.of(mapper.apply(get()));
}

Our strategic design decision is to keep the API surface area small. We prefer composability over syntactic additions / syntactic sugar. Another design goal of Vavr is to align to the Scala standard library. If the method isn't there, we also will not introduce it most probably (module type system tweaks in Java).

Why the discrepancy between Optional and Try in this case?

Vavr aligns tightly to Scala because Scala evolved over years and is well thought through. Java's API seems artificial. Take a look at java.util.Optional.of(T) for example, which throws if the argument is null.
Regarding your question: yes, indeed, mapTry is an extra method that should not be part of our API. We added it because of a problem introduced by inheritance. Try, Option and Either extend Value. Value has map(Function). Because Java does have checked exceptions, we couldn't simply overwrite Value.map(Function) with Try.map(CheckedFunction). As a workaround, we added Try.mapTry(CheckedFunction).
Btw - we already solved that problem. As part of Vavr 1.0.0, we deprecated Value. The new type at the top of the Vavr hierarchy will be ìo.vavr.Iterable, which only has one abstract method: it overloads iterator()` by returning Vavr's rich Iterator.
That way Try may define its own method map(CheckedFunction) and we are able to remove mapTry. This is the best we can do in Java.
We had to find similar workarounds in some types of our collections. The reason is that Java's type system is relatively simple and we library developers are very restricted in what can be expressed on the type level.
I hope this helps a bit.
Thx!


Support encoding body as binary

Link to issue this resolves
https://github.com/fnproject/fdk-python/issues/107

What I did
Trying to remove string encoding on response_data when body is instanceof bytes

How I did it

How to verify it

Build the related function with the test FDK

Example of README function output :


Double quotes inside subexpression

Using the String.Replace() Method inside a subexpression to remove (replace with "") parts of a string gives unexpected results.
For me, this looks related to #3039 and #4543
Steps to reproduce
"foo $( "bar".replace("bar", "") )"

Expected behavior
foo

Actual behavior
In newer PS versions, the command doesn't even work, in older versions there are different error messages about missing ending characters (like '"' and ')')
Environment data

PS C:\> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.2.2
PSEdition                      Core
GitCommitId                    6.2.2
OS                             Microsoft Windows 10.0.18362
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

The line works though, if you replace the empty double quotes with single quotes, or add a third (or even fourth) double quote:
C:\> "foo $( "bar".replace("bar", '') )" # single quotes
foo
C:\> "foo $( "bar".replace("bar", """) )" # three double quotes
foo
C:\> "foo $( "bar".replace("bar", """") )" # four double quotes
foo

Side note: to replace the string with one doublequote (without escaping or putting it inside single quotes) you need 7 double quotes in your replace() call:
C:\> "foo $( "bar".replace("bar", """"""") )" # seven double quotes
foo "

it is badly subexpression  ：
"foo $( "
+

bar
+

".replace("

+
bar

+
", "

+
") )"

Already commented about the cause of this in #10045 but comment here explains this observed behaviour (for those who like detailed explanations).
The problem appears to be that simple expandable strings are not treating an embedded subexpression as an independent syntactic context. Any pair of double quotes ("") anywhere within the string (even inside a subexpression) is assumed to represent an embedded singular double quote (") and is replaced, apparently BEFORE evaluating (or even parsing) the subexpression. This explains the observations. Demonstration follows (the replaced paired "s are highlighted between !!):
"foo $( "bar".replace("bar", !""!) )" # should be syntactically correct (ignoring the !s)

becomes
"foo $( "bar".replace("bar", ") )" # definitely not

and
"foo $( "bar".replace("bar", '') )" # single quotes so no substitutions

"foo $( "bar".replace("bar", !""!") )" # three double quotes
becomes -> "foo $( "bar".replace("bar", "") )" # looks better

"foo $( "bar".replace("bar", !""!!""!) )" # four double quotes
becomes -> "foo $( "bar".replace("bar", "") )" # ok

and the final example
"foo $( "bar".replace("bar", !""!!""!!""!") )" # seven double quotes
becomes -> "foo $( "bar".replace("bar", """") )"
# so when the subexpression is evaluated the replacement string is "!""!" i.e. a single "

This is supported by the fact that 8 quotes also works. All even numbers of quotes are equivalent to the preceding odd number since the last quote in the odd sequence remains unpaired and so is not substituted.
I hope that anyone reading this is now less confused about whatever problem they have that led them to this issue page. I would recommend looking also at #10045 since it deals with a slightly different issue with (apparently) the same root cause.


Verified that issue #1067 is resolved and added documentation for load pdf functionality.

Issue #1067 about not being able to load pdf files, was verified to be working with evadb documentation pdf and a new page for loading pdf is added to the documentation.

@kslohith Can you also please fix the broken link -- https://app.circleci.com/pipelines/github/georgia-tech-db/evadb/6047/workflows/bf1f29a3-36c9-40a0-97bb-fc084621b354/jobs/43847?invite=true#step-104-375531_124
Right now, if we load a corrupted pdf, what is the behavior?

I have added a partial unit test here (not fully written) -- https://github.com/georgia-tech-db/evadb/tree/kslohith-staging

@jarulraj I tried to load a corrupted pdf file, it was successfully loaded into evadb, giving this response.
                         0 0  Number of loaded PDF: 1
But when i try to query the table, the response is empty, i think nothing was inserted.
We should modify this behaviour and give a error message when the upload itself was a corrupted pdf instead of giving a success message.
Let me know if i should take this up and fix it.


"npm run pack" doesn't pack any dependency in my package.json

It works perfect in 'dev' mode,  then I run "npm run pack" and copy the 'electron' directory under 'dist' to a new directory, but when I try to launch my app by using command "electron electron\main.js", it fails. However,  it works fine in the 'original electron'  folder when I use the same command. It seems my app still rely on the 'node_modules', I copy the whole 'node_modules' too, then it works. I guess the webpack doesn't pack my dependencies.

@rookiebulls
If you are needing to move your electron directory, then you would need to update the main field inside your package.json to the new location. This is not a direct issue with electron-vue since you modified the document structure; closing. Feel free to ask any other questions.

@SimulatedGREG
I figure out why, it's the webpack config, I thought it would pack all the dependencies by default.
Thanks for reply so fast.


Transpiler raises when passed a list of specific circuits

Environment

Qiskit version: 1.3
Python version: 3.12
Operating system: linux

What is happening?
Consider the two circuits:

and

Passing a list of both circuits to transpile fails with
ValueError: An invalid parameter was provided.

However, if I run each circuit individually they work , and a list of multiple copies of the do nothing circuit work.  A list of the second circuit fails as well.  So it appears to be an issue with a list containing one or more of the second circuit
How can we reproduce the issue?
Try transpiling circuits like those above in a list
What should happen?
It should work.  It used to work.
Any suggestions?
No response

This sounds like it could be https://github.com/Qiskit/qiskit/issues/13504. Does it remain if you disable Qiskit running in parallel (e.g. by setting QISKIT_PARALLEL=FALSE before exeuction)?


Pass TestBlockchainTime around in consensus tests

This paves the way for having the BlockchainTime API be more restrictive
(may return PastHorizonException).

bors merge


Expose header_sensitive in RequestBuilder

Similar to #492 , a server of us requires a JWT  prefix instead of Bearer , which is hardcoded in RequestBuilder.bearer_auth().
It would be convenient to implement this correctly on our own if RequestBuilder.header_sensitive() were public. Instead only .header() is public.
Is there a chance to expose .header_sensitive?

It's possible to create a HeaderValue directly, and set the sensitive flag, and pass that to .header(name, val). That's what I would recommend, I think adding a method named header_sensitive publicly could just be confusing. It's only available internally as a minor reduction in duplicate code.

The use case that isn't supported here is using a trait to add something like a RequestBuilder.jwt_auth() fluent method.
The fluent interface can't return any errors. Instead they need to go into the internal ResultBuilder.request field.
Since the Authorization header value requires some string manipulation to add the prefix, we're going to use fallible HeaderValue::TryFrom. The only way to add any conversion error to the internal error state is to call the header method with the unconverted value.
If we pass the unconverted value to the header method, then there is no opportunity to set the sensitive flag until after the Request is created.
Exposing header_sensitive (or the ability to set the internal RequestBuilder error) would support this.


No such file or directory @ rb_sysopen

I am getting this error when running ruby app.rb      please find a solution for it
pi@raspberrypi:~/waterpi-web $ ruby app.rb
/home/pi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/mqtt-0.3.1/lib/mqtt/client.rb:198:in initialize': No such file or directory @ rb_sysopen - /home/pi/waterpi-web/certs/home/pi/waterpi-web/certs/private.pem.key (Errno::ENOENT) from /home/pi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/mqtt-0.3.1/lib/mqtt/client.rb:198:inopen'
from /home/pi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/mqtt-0.3.1/lib/mqtt/client.rb:198:in key_file=' from /home/pi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/mqtt-0.3.1/lib/mqtt/client.rb:168:inblock in initialize'
from /home/pi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/mqtt-0.3.1/lib/mqtt/client.rb:167:in each_pair' from /home/pi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/mqtt-0.3.1/lib/mqtt/client.rb:167:ininitialize'
from /home/pi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/mqtt-0.3.1/lib/mqtt/client.rb:92:in new' from /home/pi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/mqtt-0.3.1/lib/mqtt/client.rb:92:inconnect'
from /home/pi/waterpi-web/boot.rb:40:in <top (required)>' from /home/pi/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:inrequire'
from /home/pi/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in require' from app.rb:3:in
'
source code-https://github.com/demirhanaydin/waterpi-web

You opened an issue for the Guard::RSpec project, but the error you're getting is from the mqtt gem, which is unrelated.
I've never used the mqtt gem, but to me it seems you just need to generate a private key file and set the path for mqtt, or disable SSL. Documentation on connecting seems to be here: https://github.com/njh/ruby-mqtt#connecting

Duplicate of njh/ruby-mqtt#76


Bluetooth: hci: Depend on SOC not BOARD

The CONFIG_BT_RPMSG_NRF53 should depend on the SOC, not the BOARD
definition. Otherwise this will break for custom boards.
Signed-off-by: Thomas Stenersen<EMAIL_ADDRESS>

it's not only needed for custom boards, but also for multiple nRF5340 boards supported in the upstream project.


Enhancement - Update Description Field

UX Enhancement:
Primary

Update description field to include 'Markdown Supported' with proper Markdown image (from official Markdown documentation).
Enhance Description field to allow for collapsing and minimizing (toggle) text to fall between max-height of 190-200pixels and full text. The user should be able to perform these actions both when the Description field is in an editable and uneditable state.
If the Description field is minimized and in edit mode, a user should be able to scroll inside of the field, if they do not want to expand the Description field to its full height.

Secondary

Move 'Write in Markdown' and 'Preview in HTML' to align properly in a form layout.

Information

UXD will be developing this off of coded prototypes.
UXD will demonstrate new abilities (on a branch) during a review session.

VC:

Verification Conditions must be met, according to the Fabric8-UX Team (https://github.com/fabric8io/fabric8-ux/issues/348)

Started here: https://github.com/mindreeper2420/fabric8-planner/tree/UX-markdown

PR https://github.com/fabric8io/fabric8-planner/pull/1239

Closing as PR merged.


Zip Archive or plain image folders

It's more useful to have images directly exported to be ready for use.
Zip archive can be used to move files somewhere else before use.

Keep in mind for concurrency when suing file system.


is_hidden always returns true when using MultiProgress

I wrote a custom Log implementation which writes log messages using ProgressBar::println.
As expected, all my log messages got swallowed up when stderr was redirected to a file. Per the docs for ProgressBar::println, this is expected because the draw target is considered hidden (not a TTY).
To work around, I tried to call ProgressBar::is_hidden from within my logger. If the method returns true, I bypass ProgressBar::println and log directly to stderr with eprintln!.
Unfortunately, this technique does not work when using MultiProgress.
I believe the culprit is here:
https://github.com/mitsuhiko/indicatif/blob/92ce4c3f9903dbcb30a087d213ca54d041f462d6/src/draw_target.rs#L125-L131
When self.kind is Remote, we simply return false. Would it make sense to recursively call is_hidden on the remote target instead?

Side note: I am able to work around by calling console::user_attended_stderr() directly

Would it make sense to recursively call is_hidden on the remote target instead?

That certainly sounds sensible, but we'd probably want to audit calls to is_hidden() to make sure the expectations line up.

That certainly sounds sensible, but we'd probably want to audit calls to is_hidden() to make sure the expectations line up.

I submitted a fix for this. There were no calls to is_hidden() to audit, so I think we are in the clear.


Build fails on Jan 28th (and later) builds

rustc --version
rustc 1.0.0-nightly (c5961ad06 2015-01-28 21:49:38 +0000)
cargo build --release
fails with error:
src/racer/util.rs:81:13: 81:38 error: use of unmarked library feature
src/racer/util.rs:81 let s = std::rt::backtrace::write(&mut m)

replacing body get_backtrace with  return "backtrace".to_string(); helps and autocomplete works, but it is hardly a solution.

The underlying problem seems to be that std::rt::backtrace is missing a stability attribute. Commenting out get_backtrace in util.rs and the single call to it in nameres.rs:756 shouldn't have any ill effects, as it its only used for debug output.

Thanks. That is what I basically did :+1:


Check with MSRC if OE emails should have have a prefix

I am not sure how the MSRC team responds to emails. Our current guidelines simply say that any security reports should go to<EMAIL_ADDRESS>but per #780, we might want to update this with more specific instructions.
/cc @anakrish

Confirmed with MSRC that the posted guidelines is the recommended protocol for handling security issues.


Resolve mongo connection issue

What kind of change does this PR introduce? (check one with "x")
[x] Bugfix
[] Feature
[] Code style update (formatting, local variables)
[] Refactoring (no functional changes, no api changes)
[] Other... Please describe:

Fixes: #
What is the current behavior?
Gives error: topology is closed
What is the new behavior?
Gets connected to mongoDB without any issues
mongodb.zip

Hi! Can you please create a branch that only includes the changes you're suggesting for MongoDB? Right now, you're also adding a whole bunch of other activities.

Yes Sure


Initial set of operator changes for helm management repo

IBM helm management repo operator

@gyliu513 , I have onboarded this service with test-infra, I don't see the tide builds are getting started. Could you help me understand what the issue is?

/retest

/retest

/retest


Polysharp didn't work for me

I created Console App for .NET Framework 4.7.2 and added to the project file 11.0. I installed the PolySharp package .I don't see that it produces some source generators files and i got the exceptions from the compiler.. will be appreciate for advise  what i did wrong. I didn't see the Polysharp in my References list but there is packages.config file and there is a reference there. I attached zip file with a solution. There are 2 projects in solution. Please look on ConsolePolysharp2. Thank you!
ConsolePolySharp.zip

The issue is you're using a legacy-style .csproj project with the package.config configuration. This causes the source generators to not run correctly, which is why you're seeing those errors. You need to migrate to an SDK-style .csproj project 🙂
To clarify: you don't need to change framework or anything, you can keep using .NET Framework 4.7.2 if you want. But you will need to update the .csproj file to use the new format, so that the generators can run.

The issue is you're using a legacy-style .csproj project with the package.config configuration. This causes the source generators to not run correctly, which is why you're seeing those errors. You need to migrate to an SDK-style .csproj project 🙂
To clarify: you don't need to change framework or anything, you can keep using .NET Framework 4.7.2 if you want. But you will need to update the .csproj file to use the new format, so that the generators can run.

@Sergio0694 This is a very important point since .NET Framework projects are still created with legacy-style .csproj files by Visual Studio. I think this should be mentioned in the readme.

Yeah that's a good point, I'll add a warning about this in the readme! 🙂