text:
THRIFT-3876 haxe js/nodejs client

only http transport is supported
testing of browser js made via phantomjs
tutorial of browser js uses haxe-php web server for files delivery
default implementation of haxe's HttpClient reimplemented/patched:

browser js implemented to support binary encoding (avoid any utf8 conversions)
for nodejs implemented non-async client (via ChildProcess.spawnSync and it's slow)

makefile for tests updated to avoid sleeps (it kills server when client stops)

Good work!

Some of the added files lack the ASF Header. Not sure about package.json, but phantom-js-client.test.js should have one.

It does seem like this could be refactored into a base implementation holding the common code and then an implementation per #ifdef branch that was introduced to clean it up.

@Jens-G

phantom-js-client.test.js should have one.
ASF header has been added

Was this set of changes tested with "make cross" as part of the build?

@Jens-G any thoughts on what needs to happen next?

After I read that the cross tests doesn't need to be made because "it doesn't work anyway" my enthusiasm for this patch was basically send to zero . First, this is a plain lie. Second, when I read such statements it gives great insight into the attitude of the contributor. I'm not willing to even look at it anymore under that premise.

I don't care either , I had the particular issue and shared fix for the issue
I do not have time to create JIRAs for unrelated stuff nor to implement cross-platform things .
of course even asking me todo so I'm seeing for the first time in my life and I have never seen in any other foss projects
though, any maintainer have all the rights todo anything he wants

So I agree that changes like this need representation in the cross test.  It would be nice if folks could patch up any ill will and move this forward.

authors: Jens-G jeking3 oprudkyi

repo: apache/thrift


text:
THRIFT-4535: XML docs; code cleanup (tabs->spaces; String->string)

As discussed in THRIFT-4535 we'd like to go back to just one .NET library that supports .NET framework (including Mono) AND .NET Core.
As I had to learn about the library before doing some actual changes, I updated the files to contain proper XML comments (which can be used for IntelliSense) and I did some code cleanup. I did NOT change any code.
Feel free to close this PR, if you prefer to not have such a cleanup commit.
/cc @Jens-G @jeking3
PS: I'll submit actual code changes once this PR is merged or rejected.

Disclaimer: I did not look at every line and I did not look into the AppVeyor failure.
Aside from one minor thing everything that I skimmed through LTGM, and my code still compiles :-) so I'm basically fine with.

The last AppVeyor failure seems to be unrelated:

The following tests FAILED:
30 - HaskellCabalCheck (Failed)

I fixed the license header in another commit so we'll see if the build succeeds now.
Do you want me to squash the commits or will you do that via Github if you decide to merge the PR?

Squash please.

source: gharchive/pull-request

license_type: permissive

license_source: github-api


text:
THRIFT-4272:  warnings in glibc library

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[ ] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, add  [skip ci] at the end of your pull request to free up build resources.

Could you try a rebase/push please?

@Jens-G , done.

.

.


text:
vendor lib/go-* for Go components

#2380 was specifically for traffic_stats, but the same issue affects other Go components:  traffic_ops, traffic_monitor, and grove.
When the build for each component is done, the top-level for that component is copied to the RPM SOURCE directory.  Then go get grabs any missing dependencies using the import paths in the source.   In the case of lib/go-*,  the master branch of apache/incubator-trafficcontrol is pulled from github.
It should instead be using the lib/go-* directory from the user's workspace.  traffic_stats does this using a symbolic link in the vendor directory and cp -L ... in the traffic_stats.spec file.
The same strategy needs to be used for the remaining Go components unless someone can come up with a better method...

turns out I was mistaken -- some testing by introducing a syntax error in my local lib/go-log showed it was picking up the correct lib for all 3..    Closing.

source: gharchive/issue

repo: apache/trafficcontrol


text:
Go DS API needs helpful error message when creating a DS in a CDN without DNSSEC keys

If you have DNSSEC enabled on your CDN but the DNSSEC keys have not been generated for your CDN yet, it is impossible to create Delivery Services for that CDN. However, the error message returned when trying to create the Delivery Service is just "Internal Server Error".
In this case the error message returned to the user should state that the CDN has DNSSEC enabled but no DNSSEC keys yet, which prevents the creation of a new Delivery Service. IMO this should be a 400 Bad Request rather than a 500 Internal Server Error.

There appears to be a workaround for this so I've added the minor label. The workaround is to just make sure the CDN has DNSSEC keys right @rawlinp ?

I wouldn't really call it a workaround because there's no information given to the user to indicate that the issues is the CDN's DNSSEC keys, but I would agree it's minor.

@rawlinp - it's a super secret workaround :)

We now have a helpful error message for this issue.


text:
TS-4050 - Trafficserver is crashing when buckets=0 is configured in cache_promote plugin

Fix trafficserver crash when buckets=0 is configured in cache_promote plugin and Set buckets default value to be 10

Hmmm, I'll have to review this further, but there seems to be a lot other changes here in this PR, unrelated to this actual fix. Why is that? Why is so much code changed or rearranged?

Closing this. I will open new pull request with actual changes

authors: meeramn zwoop

repo: apache/trafficserver


text:
is_stale_cache_response_returnable: verify HTTP method

This adds the method verification check currently in
is_cache_response_returnable to is_stale_cache_response_returnable. That
is, if an object is being requested by the client and the object is in
our cache, we can only reply with the cached object if the method of the
incoming request is the same as the method of the request for the cached
object.

This PR came out of a discussion with @serrislew about the check here:
https://github.com/apache/trafficserver/blob/master/proxy/http/HttpTransact.cc#L1958
We shouldn't build and reply with that response if the method of the incoming request doesn't match the method of the request that resulted in the cached response.

If the stale object method is GET, will we still be able to serve HEAD from cache?

[approve ci]

[approve ci]

[approve ci fedora]

[approve ci fedora]

[approve ci fedora]

[approve ci rockylinux]

authors: bneradt mlibbey


text:
Fixes Issue #7824 - The strategies.yaml parser can incorrectly interpret

Fixes Issue #7824 - The strategies.yaml parser can incorrectly interpret the YAML elements in the host protocol in both core and the parent_select plugin.

Cherry-picked to v9.2.x

authors: jrushford zwoop


text:
[Bug] Auto scheduler Cannot find any valid schedule

Hello
I'm new to TVM and I'm currently learning AutoScheduler, and when I run auto_scheduler_matmul_x86.py this script, I get the following problem.https://tvm.hyper.ai/docs/tutorial/ops_AutoScheduling

I'm using windows11 and the version of TVM is 0.12.This is my python environment.

I'm guessing it's a matter of permissions, when I open it with admin privileges, or the same result, can anyone help me.

It appears there's a permission issue. Additionally, the subsequent error indicates no valid schedule in your log file. You can check your log file manually.
Given its extensive use and deployment, I suggest exploring metaschedule.

authors: xifengT yongwww

repo: apache/tvm


text:
[Bug] [CPP RPC] custom_addr not properly quoted in JSON string

I happened to need custom_addr option in cpp rpc. However, custom_addr_ does not seem properly quoted in the JSON string that is sent to the tracker, resulting in JSON parsing failure in the tracker.
In particular, there are 3 occurrences in apps/cpp_rpc/rpc_tracker_client.h - line 88, line 116 and line 168.

thanks for reporting @lhez, @rkimball will take a look

You are correct. We need to add " around the custom_addr since it is a string.

Here is a quick fix https://github.com/apache/tvm/pull/9192

authors: areusch lhez rkimball


text:
[tir] Add fake support for 'call_tir' op

Allow TVMScript to contain T.call_tir(...) calls. This is a small step toward a complete implementation of the 'call_tir' functionality.  The op does nothing, and its arguments are not evaluated at runtime.

@tvm-bot rerun

Hi @cconvey,
What's the reason we can't implement this now as an Allocate and a Call?

What's the reason we can't implement this now as an Allocate and a Call?

Thanks for the idea @Mousius !  Would you mind elaborating?
It's entirely possible your idea makes sense, but I'm new to this area of the code, so some more detail would be helpful.

What's the reason we can't implement this now as an Allocate and a Call?

Thanks for the idea @Mousius ! Would you mind elaborating?
It's entirely possible your idea makes sense, but I'm new to this area of the code, so some more detail would be helpful.

I believe this is implementing the same call_tir as:
https://github.com/apache/tvm-rfcs/pull/89/files#r949982699
Which would mean that for a short-term solution, instead of a no-op, you could rewrite it to allocate the buffer, here's my pseudo-code:
int size_of_output = FindSizeOfRet(prim_func->ret_type); // Can't remember a good function off the top of my head
Var tmp_buffer("tmp_buffer");

// These need to be injected before the call somehow, unsure how?
add_before = SeqStmt([
   Let(tmp_buffer, decl_buffer(size_of_output)),
   Call(prim_func, ...original args..., tmp_buffer);
]);

return tmp_buffer; // Replace Call with a the buffer?

The reason I suggest this approach is that provided you rewrote the original call_tir site to use the allocated buffer, that'd mean you could implement all your features with the "allocation and destination passing" strategy instead of having to land them without functionally?

Hey @Mousius .  There's a couple of intended differences between the TIR-level call_tir functionality being developed here and the Relax-level call_tir functionality in the Relax proposal.

Could occur within a PrimFunc, rather than being in the dataflow block of a Relax function.
Doesn't perform any allocation of the destination tensor, nor assume that the TIR function uses destination-passing style.

This could be represented at the TIR level using either builtin::tvm_call_packed, builtin::tvm_call_cpacked, or builtin::call_extern, but those would have their own drawbacks.

tvm_call_packed uses a name lookup to the TVM registry.  The T.call_tir would not have this level of indirection.  In addition, the tvm_call_packed is implemented using by resuing a local stack arguments.  The T.call_tir must be extractable into a separate context (e.g. for pipelining across threads), and cannot reuse the same stack.
tvm_call_cpacked avoids the name lookup of tvm_call_packed, but still has the stack-based arguments.
builtin::call_extern comes closest to what we  allows calls into functions defined outside the IRModule (e.g. calls to printf).  The T.call_tir would be restricted to making calls within the same IRModule.  That is, it would not allow the fallback to llvm::Function::ExternalLinkage that is used for call_extern.

Ideally, we'd want something that has the following properties

Only allows calls within the same IRModule.
Can occur within a PrimFunc as the result of lowering a single operator.
Operates in terms of a tir::Buffer, to allow it to occur at an earlier stage of TIR lowering, but avoid breaking passes that occur in-between.
Can have compile-time validation of the shape of a buffer region being passed by the caller, and the expected shape in the PrimFunc::buffer_map of the callee.

We're in the process of seeing how much of a TIR-level T.call_tir implementation could be implemented without introducing new TIR concepts, sketching out what changes would need to be proposed by RFC.

Ahh, sorry @Lunderberg, two things named the same and all that 😸 in which case would it be useful to map this to call_extern for now and add validation against the IRModule later? I'm just wondering if we can implement something so that you can see it tested and working 🤔

@tvm-bot rerun

@cconvey I think this has been superseded, can we close?

Closing for now, re-open if necessary :smile_cat:

authors: Lunderberg Mousius cconvey


text:
[Unity][Pass] Fixpoint simplification

This PR implements a simple simplification pass that applies various other simplification passes until fixpoint (i.e., apply them until the module stops changing). This probably should not be part of the standard compilation, since it has the potential to repeat passes many times, but I doubt that we would be in danger of having it happen that much. I am certain that it terminates, since all of the passes included either make the module "more simplified" or leave it unchanged if they cannot find any opportunities to make changes (i.e., no pass has the potential to undo the work of other passes). Nevertheless, the pass includes an iteration limit in case passes are later added that in some situations do not converge (not recommended).
Incidentally, it also fixes a bug that I happened to discover in the binding canonicalization pass: It did not handle the case of a non-dataflow var being bound to a dataflow var and gave a segfault if it was encountered.
A few points of discussion:

I wasn't sure if constant folding should be included in this. I am not sure new opportunities for constant folding could arise with these passes, though maybe it's possible (if we could come up with an example).
There are many normalizations happening between each round but I encountered a bug when I tried to go function-by-function (StructInfo would not update correctly), so this manner of implementation ensures fewer headaches. Passes all assume that the input has been normalized so not doing it every time has the potential to cause unexpected errors.
Are there more simplifications we would like to throw in here? Lots of little passes like the dataflow block output folding behavior would now be a lot more feasible to just throw in here and see some use.

Looks like a spurious/flaky test failure

@tvm-bot rerun

All of this should be doable in a single execution of a sequence of optimizations.  If we don't get it. we should examine our passes for deficiencies.  Doing fixed point transformations is (IMO) a warning sign to look at what we're missing that we're trying to make up for with repeated iterations.
Even in the pattern matching pass (where repetition is justifiable) we should have a max number of iterations to perform.

It's a tradeoff between increasing the complexity of the individual passes to handle more cases versus rerunning existing passes. I agree that running to fixpoint is a bit of a lazy "shotgun approach" but it could be useful in that it would allow for easily throwing in new passes to handle small transformations.

I disagree with this approach.  This is not the right way to compose transformations.

Sure.  I'll be out on vacation in the next 2 weeks, but I'd be happy to propose/work on improvements after I'm back.

I'll split the bugfix into its own PR then and we can leave this open for further discussion.

The specific motivation for this one has been superseded by later PRs like #15791 so there is no reason to keep this around.

authors: kparzysz-quic slyubomirsky


text:
[YUNIKORN-2444] Create user guide for the event system

What is this PR for?
User guide to the event system & history tracking.
What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[ ] - Feature
[x] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2444
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

I created https://issues.apache.org/jira/browse/YUNIKORN-2512 to address event.requestCapacity. After that is done, I'll update the docs again.

repo: apache/yunikorn-site


text:
[HOTFIX][ZEPPELIN-1656] z.show in Python interpreter does not work

What is this PR for?
There have been reports of #1534 causing the python interpreter to always show an error because z is not being set. As it turns out this is a result of improperly handling the case when matplotlib isn't found when initializing the interpreter.
What type of PR is it?
Bug Fix
What is the Jira issue?
ZEPPELIN-1656
How should this be tested?
Run any simple python paragraph.
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@bzz please test ASAP!

Will test right now and post back

Tested with and without matplotlib - both cases work fine!
Also
%python

a = 'a'
print "{}".format(a)

works as expected as well!
👍  thank you for prompt fix, looks awesome to me, will merge to master as hotfix asap, if there is no further discussion

authors: agoodm bzz

repo: apache/zeppelin

license_source: bigquery


text:
[ZEPPELIN-4600] Socket connection between zeppelin server and interpreter is not closed properly for master branch

What is this PR for?
Fix the stability problem of zengin for long time (7day) loop working ,
release the thread pools and socket connection when the interprether is closed
What type of PR is it?
[Bug Fix ]
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-4600
How should this be tested?

Code build passed

Screenshots (if appropriate)
Questions:

Does the licenses files need update?No
Is there breaking changes for older versions?No
Does this needs documentation?No

@tylerba-f Could you resolve the conflicts ?

conflicts was finished

authors: tylerba-f zjffdu


text:
[ZEPPELIN-4723] Enable HTTP security headers by default

What is this PR for?
Zeppelin when installed with default configuration options doesn't enable the common web application security headers, e.g. zeppelin.server.xframe.options,  zeppelin.server.xxss.protection, zeppelin.server.jetty.name, zeppelin.server.xcontent.type.options. This leaves the Zeppelin installation vulnerable.
What type of PR is it?
Improvement
Todos

Discuss HSTS header config (zeppelin.server.strict.transport) which if enabled requires TLS to be configured for Zeppelin to work

What is the Jira issue?

ZEPPELIN-4723

How should this be tested?

Below headers can be verified with received HTTP response
Server:
X-Content-Type-Options: nosniff
X-FRAME-OPTIONS: SAMEORIGIN
X-XSS-Protection: 1; mode=block

Here is Travis test run link which passed: https://travis-ci.org/github/krishna-pandey/zeppelin/builds/670946421
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@prabhjyotsingh @jongyoul @Leemoonsoo @zjffdu
Can you please help review this? Thanks.

@alexott It's happening as we speak at https://travis-ci.org/github/krishna-pandey/zeppelin/builds/670946421, also I will verify locally. Thanks.

Great! Thank you!
Can you add the link to build to the description of PR? And please update docs...

@alexott made the suggested changes, please review. Thanks.

@krishna-pandey it looks good for me, but the question is following - we updated template for configuration, not configuration itself. If user will just download & unpack, these changes won't be applicable, because zeppelin-site.xml.template isn't copied into zeppelin-site.xml.  Does it make sense to update source code with defaults as well?
@prabhjyotsingh WDYT?

Agree with @alexott We should update ZeppelinConfiguration as well

Discuss HSTS header config (zeppelin.server.strict.transport) which if enabled requires TLS to be configured for Zeppelin to work
I would love to see this in Zeppelin. Of course this should be off in default.

Can we completely suppress the zeppelin.server.jetty.name header in a production mode?

I think that we can create a followup ticket for HSTS - usually the introduction of TLS will require special build instructions, or pre-generated keys/certificates (bad idea)...

Updated ZeppelinConfiguration with default values as suggested. Verified the same reflecting locally in absence of zeppelin-site.xml and zeppelin-env.sh.

Also, Travis just finished running tests and build passed with last commit. (https://travis-ci.org/github/krishna-pandey/zeppelin/builds/671511015)
@alexott can you please review?
@Reamer IMHO not sending Server version at all or leaving it blank doesn't make much difference as everyone knows there must be a Server sending the response. A determined attacker can figure out other way to find Server details or will fire everything he had anyway.

authors: Reamer alexott krishna-pandey zjffdu


text:
[BUG]Orchestrator mysql xtrabackup backup restore failed on Azure AKS

Describe the bug
the restored cluster information is empty
k8 version
Kubernetes: v1.27.9
KubeBlocks: 0.8.3-beta.20
kbcli: 0.8.3-beta.17
WARNING: version difference between kbcli (0.8.3-beta.17) and kubeblocks (0.8.3-beta.20)

To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
1.create cluster
helm upgrade orc-80-proxy ./addons/mysql-cluster --install -n mysql-ha \
--set orchestrator.enable=true  \
--set orchestrator.serviceReference.namespace=mysql-ha \
--set orchestrator.serviceReference.port=80 \
--set orchestrator.serviceReference.endpoint=orc-cluster-orchestrator-orchestrator  \
--set proxysql.replicas=1 --set proxysql.enable=true

2.create backup
k8 cluster backup orc-80-proxy --name mysql-backup-policy-template --method xtrabackup --policy orc-80-proxy-mysql-backup-policy -n mysql-ha
Backup mysql-backup-policy-template created successfully, you can view the progress:
        kbcli cluster list-backups --name=mysql-backup-policy-template -n mysql-ha

3.backup status
         k8 cluster list-backups --name=mysql-backup-policy-template -n mysql-ha
NAME                           NAMESPACE   SOURCE-CLUSTER   METHOD       STATUS      TOTAL-SIZE   DURATION   CREATE-TIME                  COMPLETION-TIME              EXPIRATION
mysql-backup-policy-template   mysql-ha    orc-80-proxy     xtrabackup   Completed   1463123      10s        May 17,2024 14:05 UTC+0800   May 17,2024 14:05 UTC+0800

4.create restore cluster
k8 cluster restore orc-80-proxy-bp --backup  mysql-backup-policy-template -n mysql-ha
Cluster orc-80-proxy-bp created

5.see error
 k get cluster orc-80-proxy-bp -n mysql-ha
NAME              CLUSTER-DEFINITION   VERSION   TERMINATION-POLICY   STATUS   AGE
orc-80-proxy-bp   mysql                          Delete                        95s

the cluster status is empty!
6. see pod
kubectl get pod -l app.kubernetes.io/instance=orc-80-proxy-bp -n mysql-ha
No resources found in mysql-ha namespace.

there is no pod about this cluster!
7. see events
k8 cluster list-events orc-80-proxy-bp -n mysql-ha
NAMESPACE   TIME                         TYPE     REASON                  OBJECT                    MESSAGE
mysql-ha    May 17,2024 14:07 UTC+0800   Normal   PreCheckSucceed         Cluster/orc-80-proxy-bp   The operator has started the provisioning of Cluster: orc-80-proxy-bp
mysql-ha    May 17,2024 14:07 UTC+0800   Normal   ApplyResourcesSucceed   Cluster/orc-80-proxy-bp   Successfully applied for resources

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

BACKUP has required annotations to restore a cluster. But it still failes on re-store, with post-hook error

repo: apecloud/kubeblocks-addons


text:
Removes APEL Server from APEL REST Docker image and adds file for docker-compose deployment

This PR removes the apel server from this docker build, requiring the launch of a separate container for the APEL server.

it removes the conf files and scripts that were only included for the server
it removes server only requirement (python-daemon)
alters on entry configuration of docker image, as conf files are now mounted into the REST container
adds configuration files, previously created on entry, to the git repo
moves existing conf files / scripts needed for docker build to docker/
renames cloud-extra.sql to extra-cloud.sql as the new deployment method deploys schema files in alphabetical order
adds docker-compose.yaml and env files so the system can be deployed with docker-compose
updates documentation to reflect new docker-compose deployment

This is quite a large PR that seems to mix at least two, possibly three, features together (removal of APEL server, docker-compose stuff, and maybe config changes). Can it be split into separate pull requests using a reasonable amount of effort? From a glance, the commits seem to be separate at least.
A branch, which should end in a PR, should be a single feature to make the branch history clearer and for ease of reviewing. Also, need more context and "why"s in PRs and commit messages. I think I agree with most of the stuff in this guide.

I have split the first commit up and compressed the removal of file commits.
I've also tried to add why the changes have been made in the commit messages

The other options is a docker image with the schema files pre loaded/empty database pre created, but I'm not sure that's better because

It means we have another docker image to support
All the schemas you want to have in the image to be known at build time. Then if you wanted a subset of the databases you'd be stuck, you'd just have to ignore the unused databases
I think it would involve a git repo for just the schema files so the docker could be built automatically

I agree the alphabetical method seems quite fragile but I think it's the best of the two options, I'll add numbers and a README.md documenting this feature
I'll make the documentation changes as well

grammar change made

Travis has run, but test results don't seem to be getting back.

yeah... maybe when I do the rebase it will trick travis into updating

This needs rebasing and the conflicts resolving. Need any help with that?

I should be able to do the rebase/handle the merge conflicts. It's waiting on the APEL server docker image PR.

No longer dependant on apel PR#124 due to that functionality being added to the Indigo DataCloud Fork of APEL (see PR#1).
I'll rebase this branch and fix the conflicts.

This has been rebased and is ready for merging, but the rebase dismissed your review.

authors: gregcorbett tofu-rocketry

repo: apel/rest


text:
More accurate since tag for this class

To avoid confusion when reading the source and debugging, etc.

Thanks Dima!

repo: apereo/cas


text:
Command authentication has no action

Describe the bug
Command authentication has no action,
To Reproduce

setup hy2 server
connect to hy2 server

Expected behavior
A command added to auth: section be executed upon each user correct authentication
auth:
  type: userpass
  password: ...
  command: /opt/app/hy2/hy2.sh

the hy2.sh
#!/bin/bash

echo "${@}" >> hy2.txt

echo $2
exit 0

and having these  values as arguments for the command

IP
username (id)

{"addr": "a.b.c.d:46141", "id": "shakiba", "tx": 0}

Logs
no error in either sides
server
2024-05-04T08:31:23+02:00	INFO	server mode
2024-05-04T08:31:23+02:00	INFO	server up and running	{"listen": ":3443"}
2024-05-04T08:31:24+02:00	INFO	update available	{"version": "v2.4.3", "url": "https://github.com/apernet/hysteria/releases", "urgent": true}
2024-05-04T08:31:43+02:00	INFO	client connected	{"addr": "a.b.c.d:46141", "id": "shakiba", "tx": 0}

client
2024-05-04T10:01:42+03:30	INFO	client mode
2024-05-04T10:01:43+03:30	INFO	connected to server	{"udpEnabled": true, "tx": 0, "count": 1}

Device and Operating System
Server Debian 11
Client Manjaro
Hy2 Version
both sides
Version:	v2.3.0
BuildDate:	2024-03-10T06:38:38Z
BuildType:	release
CommitHash:	02baab148a7cb29f7f97696bfcbe56dbb8801f0a
Platform:	linux
Architecture:	amd64

Additional context
is the document Command authentication up to date ?
When using command authentication, the server will execute the specified command with the following arguments when a client attempts to connect:

Change type: userpass to type: command

authors: shakibamoshiri tobyxdd

repo: apernet/hysteria

license: MIT


text:
Add a minimum IAM policy for VPC

Discussed in #602 .

thanks!

authors: outsideris tj

repo: apex/apex


text:
Slow Resize Rate for the Chart

After creating a line chart in my page, I found that when I resize the browser, the chart would not immediately resize. The chart only re-adapted to the page when I adjusted the browser window for a while.
I viewed the source code and found some comments:
this.w.globals.resizeTimer = window.setTimeout(function () {
          _this6.w.globals.resized = true;
          _this6.w.globals.dataChanged = false; // we need to redraw the whole chart on window resize (with a small delay).

          _this6.ctx.update();
        }, 150);

However, 150ms refresh is indeed slow, for those who have high requirements for user experience and interface, even if they sometimes ignore the performance - they want the chart to adapt to the page in a timely manner. The feeling that the chart is updated only after dragging for a while is really uncomfortable. When I changed 150 into a smaller number, the using experience was much better. Therefore, I kindly suggest adding a parameter to let developers freely choose the interval of this resize function.

Changing to a feature request


text:
Mint NFT collection

How can i mint 10k NFT collection easily. Can't mint NFTs quickly. Getting error when send 3+ tx

Please contact me. I and you will deal price for 10k NFT. https://t.me/IamKhoi1402

repo: apezord/doginals


text:
Propose resource relation syntax

Allows specifying of link relation types. I wasn't sure about adding examples in the example folder since the given example in the spec is pretty straight forward.
Related to #88

Looks good except one typo

Updated.

authors: pksunkara zdne

repo: apiaryio/api-blueprint

license: mit


text:
Show clear error message if the stream requires authentication

Is your feature request related to a problem? Please describe.
When I forget to set an --auth-token I receive a long grpc error message. It's not clear what the message is about.
Describe the solution you'd like
The message should inform me that I need to authenticate with the stream.

This is now included.

repo: apibara/dna


text:
Contract test: Handle responses with no body

Discussed in https://github.com/apideck-libraries/portman/discussions/239

Originally posted by savage-alex January 14, 2022
204 is a popular status code for PUT, POST and DELETE operations and MUST not have a response body says the RFC7231
In portman contract tests it correctly omits any body or schema checks if there is no content in the and API definition for the 2xx response. What it currently does not to is add a check for response body to be empty style test such as
pm.test("response should be empty", function () {
pm.response.to.not.be.withBody;
});
Is this something that would be useful to add to ensure we are not returning bodies when the contact is saying we shall not?

@savage-alex It works very well, Thank you @thim81

repo: apideck-libraries/portman


text:
Fix test status code in contract test #238

A tiny fix, Linked to #238

@AliKhalili I tested your PR, it does the trick.
I'm going to see if I can add a test so can have it covered in the future.

@AliKhalili I'm not a coder by nature and I don't fully understand how your code change solved the issue.
before:
const statusCodeSetting = optStatusCode  as StatusCode
after:
const statusCodeSetting = { ...optStatusCode } as StatusCode
Would you mind enlighten me what is the difference?

@thim81
Your codes are very well implemented and I like your implementation. I learn many things from Portman source code.
At this line of code you create a new statusCodeSetting variable but it just a new reference to the old variable optStatusCode , so when we changed the statusCodeSetting, it also changes the value of optStatusCode . for solving this issue we need to create a new object by copy the value of all properties one by one which we can do this by spread syntax.
When a variable refers to an object (including arrays), the "value" is a reference to the object.
https://dmitripavlutin.com/static/21075d0959751add1730239d141bbe87/42163/references-2.png

Thanks for explaining and I now see that indeed it was a "reference".
I must have missed that + I think we might run into some more referenced variables.
Thanks again for your PR


text:
feat(utils): add purgeLocalStorage method

Handles one of the requests in #1030
There is now a utils.purgeLocalStorage method you can use to cleanup the local storage folder, if needed, without needing to run the CLI

Some..cookie test seems to have failed? @B4nan should I just re-run the CIs or?

I guess we should disable that one, its failing too often.

repo: apify/apify-js


text:
HandleAuthorization called twice per request from Mixer

As of Istio 1.0 prerelease builds, the HandleAuthorization() function is being called twice in Mixer builds. It appears this is related to rules changes.
See: https://github.com/istio/istio/issues/6804

There is a suggestion for the rules in the above issue discussion to change the rule to a match like:
match: context.reporter.kind == "inbound" && destination.labels["app"] == "backend"

However, the most recent Istio snapshot (https://github.com/istio/istio/releases/tag/1.0.0-snapshot.0) doesn't support context.reporter.kind. Marking this issue as pending.

repo: apigee/istio-mixer-adapter


text:
Deserialization usage example

Hey, I would like to use this library to analyse the blockchain. I already have a downloaded copy, using the standard bitcoind. I assumed that I could use bitcoin::network::serialize::deserialize to deserialize a blk*.dat file into a Block struct. Is that correct? When I try to do that, the deserializer allocates an extremely large Vec<u8> and crashes because it's out of memory.

No, the blk*.dat files are not stored in the same format as blocks are in the Bitcoin network protocol. You cannot parse them with rust-bitcoin.
Thank you for reporting this memory-DoS issue though!

Thanks for your fast response! I'll look for another way then. If you want you can close this issue.

Fixed in https://github.com/apoelstra/rust-bitcoin/commit/cdb452f79faea376034c835557ad0cfe4ccd0659

authors: apoelstra maufl

repo: apoelstra/rust-bitcoin


text:
对于密码一类配置，除了加密存储外，能否在Portal展示时进行掩码展示（即使是密文）

对于密码或类似的敏感数据的配置，在Portal上展示时，能否进行掩码展示。
即，在Portal上添加一个配置时，可以指定该配置是安全敏感的，那么在存储时可以进行加密存储（加密存储社区内已有实现），在展示时进行掩码展示（如全部显示成*）

see #3187

Duplicate of #3187

authors: JefferyCat nobodyiam

repo: apolloconfig/apollo


text:
docker拉取docker hub 1.7.0镜像_APOLLO-ADMINSERVICE不注册8080的eureka

2020-08-08 23:02:12.762  WARN 34 --- [tbeatExecutor-0] c.n.d.s.t.d.RetryableEurekaHttpClient    : Request execution failed with message: java.net.ConnectException: Connection refused (Connection refused)
2020-08-08 23:02:12.762 ERROR 34 --- [tbeatExecutor-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_APOLLO-ADMINSERVICE/e0033a57e006:apollo-adminservice:8090 - was unable to send heartbeat!

com.netflix.discovery.shared.transport.TransportException: Cannot execute request on any known server
        at com.netflix.discovery.shared.transport.decorator.RetryableEurekaHttpClient.execute(RetryableEurekaHttpClient.java:112)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.sendHeartBeat(EurekaHttpClientDecorator.java:89)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$3.execute(EurekaHttpClientDecorator.java:92)
        at com.netflix.discovery.shared.transport.decorator.SessionedEurekaHttpClient.execute(SessionedEurekaHttpClient.java:77)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.sendHeartBeat(EurekaHttpClientDecorator.java:89)
        at com.netflix.discovery.DiscoveryClient.renew(DiscoveryClient.java:846)
        at com.netflix.discovery.DiscoveryClient$HeartbeatThread.run(DiscoveryClient.java:1399)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)

2020-08-08 23:02:12.777  INFO 34 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Disable delta property : false
2020-08-08 23:02:12.778  INFO 34 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Single vip registry refresh property : null
2020-08-08 23:02:12.778  INFO 34 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Force full registry fetch : false
2020-08-08 23:02:12.778  INFO 34 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Application is null : false
2020-08-08 23:02:12.778  INFO 34 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Registered Applications size is zero : true
2020-08-08 23:02:12.778  INFO 34 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Application version is -1: true
2020-08-08 23:02:12.779  INFO 34 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Getting all instance registry info from the eureka server
2020-08-08 23:02:12.784 ERROR 34 --- [freshExecutor-0] c.n.d.s.t.d.RedirectingEurekaHttpClient  : Request execution error

com.sun.jersey.api.client.ClientHandlerException: java.net.ConnectException: Connection refused (Connection refused)
        at com.sun.jersey.client.apache4.ApacheHttpClient4Handler.handle(ApacheHttpClient4Handler.java:187)
        at com.sun.jersey.api.client.filter.GZIPContentEncodingFilter.handle(GZIPContentEncodingFilter.java:123)
        at com.netflix.discovery.EurekaIdentityHeaderFilter.handle(EurekaIdentityHeaderFilter.java:27)
        at com.sun.jersey.api.client.Client.handle(Client.java:652)
        at com.sun.jersey.api.client.WebResource.handle(WebResource.java:682)
        at com.sun.jersey.api.client.WebResource.access$200(WebResource.java:74)
        at com.sun.jersey.api.client.WebResource$Builder.get(WebResource.java:509)
        at com.netflix.discovery.shared.transport.jersey.AbstractJerseyEurekaHttpClient.getApplicationsInternal(AbstractJerseyEurekaHttpClient.java:194)
        at com.netflix.discovery.shared.transport.jersey.AbstractJerseyEurekaHttpClient.getApplications(AbstractJerseyEurekaHttpClient.java:165)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$6.execute(EurekaHttpClientDecorator.java:137)
        at com.netflix.discovery.shared.transport.decorator.MetricsCollectingEurekaHttpClient.execute(MetricsCollectingEurekaHttpClient.java:73)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$6.execute(EurekaHttpClientDecorator.java:137)
        at com.netflix.discovery.shared.transport.decorator.RedirectingEurekaHttpClient.executeOnNewServer(RedirectingEurekaHttpClient.java:118)
        at com.netflix.discovery.shared.transport.decorator.RedirectingEurekaHttpClient.execute(RedirectingEurekaHttpClient.java:79)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$6.execute(EurekaHttpClientDecorator.java:137)
        at com.netflix.discovery.shared.transport.decorator.RetryableEurekaHttpClient.execute(RetryableEurekaHttpClient.java:120)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$6.execute(EurekaHttpClientDecorator.java:137)
        at com.netflix.discovery.shared.transport.decorator.SessionedEurekaHttpClient.execute(SessionedEurekaHttpClient.java:77)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
        at com.netflix.discovery.DiscoveryClient.getAndStoreFullRegistry(DiscoveryClient.java:1051)
        at com.netflix.discovery.DiscoveryClient.fetchRegistry(DiscoveryClient.java:965)
        at com.netflix.discovery.DiscoveryClient.refreshRegistry(DiscoveryClient.java:1473)
        at com.netflix.discovery.DiscoveryClient$CacheRefreshThread.run(DiscoveryClient.java:1440)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)
Caused by: java.net.ConnectException: Connection refused (Connection refused)
        at java.net.PlainSocketImpl.socketConnect(Native Method)
        at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:350)
        at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)
        at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)
        at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
        at java.net.Socket.connect(Socket.java:589)
        at org.apache.http.conn.scheme.PlainSocketFactory.connectSocket(PlainSocketFactory.java:121)
        at org.apache.http.impl.conn.DefaultClientConnectionOperator.openConnection(DefaultClientConnectionOperator.java:180)
        at org.apache.http.impl.conn.AbstractPoolEntry.open(AbstractPoolEntry.java:144)
        at org.apache.http.impl.conn.AbstractPooledConnAdapter.open(AbstractPooledConnAdapter.java:134)
        at org.apache.http.impl.client.DefaultRequestDirector.tryConnect(DefaultRequestDirector.java:610)
        at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:445)
        at org.apache.http.impl.client.AbstractHttpClient.doExecute(AbstractHttpClient.java:835)
        at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:118)
        at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:56)
        at com.sun.jersey.client.apache4.ApacheHttpClient4Handler.handle(ApacheHttpClient4Handler.java:173)
        ... 30 common frames omitted

2020-08-08 23:02:12.786  WARN 34 --- [freshExecutor-0] c.n.d.s.t.d.RetryableEurekaHttpClient    : Request execution failed with message: java.net.ConnectException: Connection refused (Connection refused)
2020-08-08 23:02:12.787 ERROR 34 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_APOLLO-ADMINSERVICE/e0033a57e006:apollo-adminservice:8090 - was unable to refresh its cache! status = Cannot execute request on any known server

com.netflix.discovery.shared.transport.TransportException: Cannot execute request on any known server
        at com.netflix.discovery.shared.transport.decorator.RetryableEurekaHttpClient.execute(RetryableEurekaHttpClient.java:112)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$6.execute(EurekaHttpClientDecorator.java:137)
        at com.netflix.discovery.shared.transport.decorator.SessionedEurekaHttpClient.execute(SessionedEurekaHttpClient.java:77)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.getApplications(EurekaHttpClientDecorator.java:134)
        at com.netflix.discovery.DiscoveryClient.getAndStoreFullRegistry(DiscoveryClient.java:1051)
        at com.netflix.discovery.DiscoveryClient.fetchRegistry(DiscoveryClient.java:965)
        at com.netflix.discovery.DiscoveryClient.refreshRegistry(DiscoveryClient.java:1473)
        at com.netflix.discovery.DiscoveryClient$CacheRefreshThread.run(DiscoveryClient.java:1440)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)

2020-08-08 23:02:13.216  INFO 34 --- [nfoReplicator-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_APOLLO-ADMINSERVICE/e0033a57e006:apollo-adminservice:8090: registering service...
2020-08-08 23:02:13.217 ERROR 34 --- [nfoReplicator-0] c.n.d.s.t.d.RedirectingEurekaHttpClient  : Request execution error

com.sun.jersey.api.client.ClientHandlerException: java.net.ConnectException: Connection refused (Connection refused)
        at com.sun.jersey.client.apache4.ApacheHttpClient4Handler.handle(ApacheHttpClient4Handler.java:187)
        at com.sun.jersey.api.client.filter.GZIPContentEncodingFilter.handle(GZIPContentEncodingFilter.java:123)
        at com.netflix.discovery.EurekaIdentityHeaderFilter.handle(EurekaIdentityHeaderFilter.java:27)
        at com.sun.jersey.api.client.Client.handle(Client.java:652)
        at com.sun.jersey.api.client.WebResource.handle(WebResource.java:682)
        at com.sun.jersey.api.client.WebResource.access$200(WebResource.java:74)
        at com.sun.jersey.api.client.WebResource$Builder.post(WebResource.java:570)
        at com.netflix.discovery.shared.transport.jersey.AbstractJerseyEurekaHttpClient.register(AbstractJerseyEurekaHttpClient.java:56)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$1.execute(EurekaHttpClientDecorator.java:59)
        at com.netflix.discovery.shared.transport.decorator.MetricsCollectingEurekaHttpClient.execute(MetricsCollectingEurekaHttpClient.java:73)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.register(EurekaHttpClientDecorator.java:56)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$1.execute(EurekaHttpClientDecorator.java:59)
        at com.netflix.discovery.shared.transport.decorator.RedirectingEurekaHttpClient.executeOnNewServer(RedirectingEurekaHttpClient.java:118)
        at com.netflix.discovery.shared.transport.decorator.RedirectingEurekaHttpClient.execute(RedirectingEurekaHttpClient.java:79)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.register(EurekaHttpClientDecorator.java:56)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$1.execute(EurekaHttpClientDecorator.java:59)
        at com.netflix.discovery.shared.transport.decorator.RetryableEurekaHttpClient.execute(RetryableEurekaHttpClient.java:120)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.register(EurekaHttpClientDecorator.java:56)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$1.execute(EurekaHttpClientDecorator.java:59)
        at com.netflix.discovery.shared.transport.decorator.SessionedEurekaHttpClient.execute(SessionedEurekaHttpClient.java:77)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.register(EurekaHttpClientDecorator.java:56)
        at com.netflix.discovery.DiscoveryClient.register(DiscoveryClient.java:829)
        at com.netflix.discovery.InstanceInfoReplicator.run(InstanceInfoReplicator.java:121)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)
Caused by: java.net.ConnectException: Connection refused (Connection refused)
        at java.net.PlainSocketImpl.socketConnect(Native Method)
        at java.net.AbstractPlainSocketImpl.doConnect(AbstractPlainSocketImpl.java:350)
        at java.net.AbstractPlainSocketImpl.connectToAddress(AbstractPlainSocketImpl.java:206)
        at java.net.AbstractPlainSocketImpl.connect(AbstractPlainSocketImpl.java:188)
        at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:392)
        at java.net.Socket.connect(Socket.java:589)
        at org.apache.http.conn.scheme.PlainSocketFactory.connectSocket(PlainSocketFactory.java:121)
        at org.apache.http.impl.conn.DefaultClientConnectionOperator.openConnection(DefaultClientConnectionOperator.java:180)
        at org.apache.http.impl.conn.AbstractPoolEntry.open(AbstractPoolEntry.java:144)
        at org.apache.http.impl.conn.AbstractPooledConnAdapter.open(AbstractPooledConnAdapter.java:134)
        at org.apache.http.impl.client.DefaultRequestDirector.tryConnect(DefaultRequestDirector.java:610)
        at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:445)
        at org.apache.http.impl.client.AbstractHttpClient.doExecute(AbstractHttpClient.java:835)
        at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:118)
        at org.apache.http.impl.client.CloseableHttpClient.execute(CloseableHttpClient.java:56)
        at com.sun.jersey.client.apache4.ApacheHttpClient4Handler.handle(ApacheHttpClient4Handler.java:173)
        ... 29 common frames omitted

2020-08-08 23:02:13.218  WARN 34 --- [nfoReplicator-0] c.n.d.s.t.d.RetryableEurekaHttpClient    : Request execution failed with message: java.net.ConnectException: Connection refused (Connection refused)
2020-08-08 23:02:13.218  WARN 34 --- [nfoReplicator-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_APOLLO-ADMINSERVICE/e0033a57e006:apollo-adminservice:8090 - registration failed Cannot execute request on any known server

com.netflix.discovery.shared.transport.TransportException: Cannot execute request on any known server
        at com.netflix.discovery.shared.transport.decorator.RetryableEurekaHttpClient.execute(RetryableEurekaHttpClient.java:112)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.register(EurekaHttpClientDecorator.java:56)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$1.execute(EurekaHttpClientDecorator.java:59)
        at com.netflix.discovery.shared.transport.decorator.SessionedEurekaHttpClient.execute(SessionedEurekaHttpClient.java:77)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.register(EurekaHttpClientDecorator.java:56)
        at com.netflix.discovery.DiscoveryClient.register(DiscoveryClient.java:829)
        at com.netflix.discovery.InstanceInfoReplicator.run(InstanceInfoReplicator.java:121)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)

2020-08-08 23:02:13.218  WARN 34 --- [nfoReplicator-0] c.n.discovery.InstanceInfoReplicator     : There was a problem with the instance info replicator

com.netflix.discovery.shared.transport.TransportException: Cannot execute request on any known server
        at com.netflix.discovery.shared.transport.decorator.RetryableEurekaHttpClient.execute(RetryableEurekaHttpClient.java:112)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.register(EurekaHttpClientDecorator.java:56)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator$1.execute(EurekaHttpClientDecorator.java:59)
        at com.netflix.discovery.shared.transport.decorator.SessionedEurekaHttpClient.execute(SessionedEurekaHttpClient.java:77)
        at com.netflix.discovery.shared.transport.decorator.EurekaHttpClientDecorator.register(EurekaHttpClientDecorator.java:56)
        at com.netflix.discovery.DiscoveryClient.register(DiscoveryClient.java:829)
        at com.netflix.discovery.InstanceInfoReplicator.run(InstanceInfoReplicator.java:121)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)

注册失败呢，是不是应该注入一个属性是eureka的地址
2020-08-08 23:04:13.290  WARN 34 --- [nfoReplicator-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_APOLLO-ADMINSERVICE/e0033a57e006:apollo-adminservice:8090 - registration failed Cannot execute request on any known server

遇到一样的问题，1.7.0版本

需要配置下eureka.service.url - Eureka服务Url

额，总算知道了，教程里漏了，需要修改ApolloConfigDB中表ServerConfig，修改其中一条记录Key为eureka.service.url，将其Value修改为Config Server的地址，默认设置的是localhost，Docker方式启动Admin Server通过localhost是访问不到Config Server的服务。

我遇到了同样的问题，但是我照着步骤去改了 eureka.service.url ，但是仍然没有解决
eureka 的 UI界面里是这样的

authors: henrybit nobodyiam zhaokejin zzq361267211


text:
AbstractMethodError exception from RealCacheKeyBuilder

Description
Setting up a query which looks like this :
mutation TrackUserActivity($input:TrackEventsInput!) {
  trackEvents(input:$input){
      __typename,
      completed
  }
}

And the expect output is something like this
{
  "data": {
    "trackEvents": {
      "__typename": "TrackEventsPayload",
      "completed": true
    }
  }
}

And my apollo client is setup like this
ApolloClient.builder()
                .serverUrl(url)
                .okHttpClient(okHttpClient)
                .normalizedCache(LruNormalizedCacheFactory(EvictionPolicy.NO_EVICTION), object : CacheKeyResolver() {
                    override fun fromFieldRecordSet(field: ResponseField, recordSet: Map<String, Any>): CacheKey {
                        val id = (recordSet["id"] as? String)
                        return if (id != null) {
                            CacheKey.from(id)
                        } else {
                            CacheKey.NO_KEY
                        }
                    }

                    override fun fromFieldArguments(field: ResponseField, variables: Operation.Variables): CacheKey {
                        return CacheKey.NO_KEY
                    }
                })

With the above setup everytime I run it, I am seeing the error below:

Any ideas what is wrong with this query.
Version
2.1.0

Hi, it looks like it is because of a bug resulting from our Kotlin conversion. To avoid it, you could go back to 1.4.x version. We will try our best to include a fix for this in the next version.

Thanks @tasomaniac, going back 1.4 now would be hard for us, as we have build lot of new stuff on top of 2.0 version.
It would be super helpful if you could prioritize this for the next upcoming release.

@ahetawal-p can you try with version 2.2.0 ?

Thanks @martinbonnin. I just tried and it resolved my issue. Thank you so much for the quick fix. ❤

authors: ahetawal-p martinbonnin tasomaniac

repo: apollographql/apollo-android


text:
Add a flag to disable the fieldsCanMerge validation on disjoint types

Given this schema:
  type Query {
    foo: PropertyValue
  }
  union PropertyValue = PropertyValueBoolean | PropertyValueInt

  type PropertyValueBoolean {
    name: String!
    value: Boolean
  }
  type PropertyValueInt {
    name: String!
    value: Int
  }

and this query:
  query GetFoo {
    foo {
      ...PropertyValue
    }
  }
  fragment PropertyValue on PropertyValue {
    __typename
    ... on PropertyValueBoolean {
        value
    }
    ... on PropertyValueInt {
        value
    }
  }

Validation fails with:
`value` cannot be merged with `value`: they have different shapes.

This is expected from reading the spec although it's not clear what the benefit of this validation is on completely disjoint types and it makes the generated models more akward to use.
Other users have bumped into the same issue in other projects like here
We could have a Gradle flag that specifically disables this validation:
apollo {
  fieldOnDisjointTypesMustMerge = false
}

More context:

Spec pull request: https://github.com/graphql/graphql-spec/pull/162
Slack discussion: https://kotlinlang.slack.com/archives/C01A6KM1SBZ/p1659536084099869

Can I give this a try?

Very sorry for the delay @pt2121 ! Thanks for looking into this. I'm reviewing the PR now, I'll follow up there.

This is now available in 3.6.0

repo: apollographql/apollo-kotlin


text:
4.0.5 patch breaks @skip and @include directives in schema stitching

After we have updated graphql-tools to 4.0.5:

@skip and @include directives broke in schema stitching.
For example we had this query:
 games(order_by: {
  stats: {
   matches: desc
  }
 }) {
  id
  name
  shortName
  rawName: raw_name
  disabled
  stats @include(if: $requestStats) {
   id
   matchesCount: matches
   usersOnlineCount: users_online
   usersPlayed: users_played
  }
 }
}

We get this:

See https://github.com/graphql/graphql-js/blob/master/src/type/schema.js#L104
Not having any directive support in addResolversToSchemas was protecting against default of no directives in mergeSchemas and adding back the default directives. Fix is within mergeSchemas to pass undefined instead of empty array when directive option is not enabled.

Should be fixed in graphql-tools-fork. Let me know if you can if you have any issues.

Any news about this issue?

this was the solution for our case.

We recently released an alpha version of GraphQL Tools (#1308) that should fix the issue.
Please update graphql-tools to next or run:
npx match-version graphql-tools next

Let us know if it solves the problem, we're counting for your feedback! :)

Rolled into #1306 .

authors: Grmiade Necmttn ekifox kamilkisiela yaacovCR

repo: apollographql/graphql-tools


text:
Mobile View Header issue

Description
In Mobile view, the user panel header doesn't contain "Hello " as appear in desktop view. Username should be dynamic according to the user email id
#Mobile view

#Desktop View

Screenshots
!important :    Need (Hello User) not Hello  because user id could be long and may not be fit in mobile view

@apoorvaron Assign this issue to me, i will fix it.

@AmanJain18 Assigned

@AmanJain18 update?

@AmanJain18 ??

@AmanJain18 ??

Ya will fix it shortly, I was busy with other work stuff...

@apoorvaron I want to work on this issue, please assign me.

@AlfiyaSiddique after this issue, there is one more requirement as the number of links in Navbar links increases , it becomes less impactful , so i want a drop down.

if you want to work on this , create a issue , if not ping me also

@apoorvaron Of course I can work on that issue, but  at which size do you want dropdown?
And also in this issue, the mobile view should Contain the hardcode text Hello User! and not the dynamic text  Hello [UserId ] for Example Hello Shorty! , right?

Yes , because other person can contain too much long email id and that doesn't fit in mobile view that's why

Of course I can work on that issue, but at which size do you want dropdown?

@AlfiyaSiddique I need a extra drop down in all views (Something like this : Buttons + extra Navbar links + Dropdown)

Okay, @apoorvaron  understood, I'll create an issue, later in that issue tell me which links you want me to add in the dropdown,

@AlfiyaSiddique for now just add dummy data

authors: AlfiyaSiddique AmanJain18 apoorvaron

repo: apoorvaron/Shorty


text:
python version and tag

Use pyproject.toml to define the Python version.
Fix final image tag

/retest

repo: app-sre/er-aws-msk


text:
schema: add resource type name constants

This was just a stupid oversight in implementing these types. Correct it
so a user has access to the names if needed.

cc @cdaylward @jonboulle

actually, I am going to just merge this, it is a trivial code cleanup.

repo: appc/spec


text:
Point cloud.atomiq.io at proper server

It doesn't appear that cloud.atomiq.io is pointed at a proper server. It should at least have a web page to describe it, or if the service isn't running, we should create it somewhere.

we can configure the main proxy of the cluster with base domain cloud.atomiq.io to default to a service serving a web page with information about this cluster (ULA, basic usage guidance, etc).

authors: ingo ndegory

repo: appcelerator/amp


text:
AppMaps can be uploaded from the IntelliJ plugin

See the VSCode implementation as a reference for this ticket: https://github.com/applandinc/vscode-appland/blob/feat/appmap-upload/src/appmapUploader.ts

a new "Upload" icon exists in the AppMap diagram
when user clicks on the icon, a "Upload? Y/N" confirmation message is displayed. When confirmed by the user, the upload proceeds
The upload action opens a new web browser window and posts the AppMap to the upload endpoint on https://app.land

Upload icon:
appmap_upload_icon.svg.txt

This has been merged already.

authors: jansorg ptrdvrk

repo: applandinc/appmap-intellij-plugin


text:
Missing py.typed file causes type checkers to ignore type annotations

Hi, thanks for the great package! I'm already using it to explore migration from verifyReceipt.
The core issue is this:
$ pip3 install app-store-server-library
$ pip3 install mypy
$ cat test.py
import appstoreserverlibrary as aslib

print(aslib.models.Environment.PRODUCTION)
$ mypy test.py
test.py:1: error: Skipping analyzing "appstoreserverlibrary": module is installed, but missing library stubs or py.typed marker  [import]
test.py:1: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports
Found 1 error in 1 file (checked 1 source file)

I believe the issue is that while this project includes type annotations (which is great), it is missing the py.typed file. That tells mypy and other type checkers to treat it as having type annotations. You can find details either in PEP 561:

Package maintainers who wish to support type checking of their code MUST add a marker file named py.typed to their package supporting typing. This marker applies recursively: if a top-level package includes it, all its sub-packages MUST support type checking as well.

or in mypy's documentation:

If you would like to publish a library package to a package repository yourself (e.g. on PyPI) for either internal or external use in type checking, packages that supply type information via type comments or annotations in the code should put a py.typed file in their package directory.

@WFT Could you please test with 0.2.1 in pypi-test when you have a chance and see if the issue is resolved?

Just checked it out. Looks like it works perfectly. This can be closed.

(Looks like the test file in the original doesn't actually typecheck due to some typos; here's one that does)
import appstoreserverlibrary.models.Environment

print(appstoreserverlibrary.models.Environment.Environment.PRODUCTION)

$ mypy test.py
Success: no issues found in 1 source file

authors: WFT alexanderjordanbaker

repo: apple/app-store-server-library-python


text:
Update werkzeug dependency to remove vulnerability

Removes werkzeug vulnerability https://github.com/advisories/GHSA-2g68-c3qc-8985

Unit tests should all pass. I don’t know what hidden implications this library has
for kicking off circle CI for tests, I think authorized committer needs to hit a button

Unit tests should all pass. I don’t know what hidden implications this library has
for kicking off circle CI for tests, I think authorized committer needs to hit a button

CircleCI tests do not show up, will look into it to see what happened.

make PR from your own fork, not main repo

CircleCI tests do not show up, will look into it to see what happened.

make PR from your own fork, not main repo

repo: apple/pfl-research


text:
Type parameters in new Mapping<TypeA, TypeB>/ new Listing<Type> are not checked

This does not throw, but should:
foo = new Listing<Int> {
  "hello"
}

In the below, both types should be checked:
foo: Listing<Int(isEven)> = new Listing<Int(this > 5)> { 1; 2; 3 }

The check should not follow when amended.
This is fine:
foo: new Listing<Int> { 1; 2; 3 }

bar = (foo) { "hello" }

This should probably be fine too:
foo = new Listing<Int> { 1 } { "hello" }

This is just another example. The addition happens and we get 23 but at what point is the type checking meant to happen. Using the IntelliJ plugin I get warnings telling me I'm doing something stupid.
foo = new Listing<YamlRenderer> {
  11
}

bar = new Listing<String> {
  22
}

output {
  value = foo[0] + bar[0]
  renderer = new YamlRenderer{}
}

output
33

A pragmatic solution would be to allow x: Listing<A> = new Listing {...}(*) but not x = new Listing<A> {...}. This would reflect the (current) reality that generics aren't reified. It would also encourage users to do the right thing and put type annotations on their properties.
(*) or x: Listing<A> = (Listing) {...}

That's less simple than you might think. The generic informs the default / IDE type checking. It's a common pattern to define Listings inline to produce something of a very different type, such as new Listing<String> { ... }.join(", ") to produce a String.

Dealing with generics is never simple. Regarding your concrete arguments, Listing.join is defined for all Listings, not just Listing<String>. And perhaps IDEs could infer the element type from the given elements. The default sounds like a problem though.

authors: bioball harryjackson holzensp translatenix

repo: apple/pkl


text:
Revert "Add setting for Kotlin package to codegen (#194)"

This reverts commit 7f404fff490e2dd883cf9683c21f6fd38fd84518.
The package is derived from the module name.
Having module com.example.Foo in Pkl
will create Kotlin package com.example.

Heads up to @sgammon: this reverts your commit, because there's already a way to control the package name.

@bioball No problem, understood. But did you see my comment about JPMS? This was added as an escape hatch for that specific issue

Ah, I see. How does a special package prefix flag change or fix this? To be clear, what's the difference between:
module my.Module

kotlinCodeGenerators {
  register("configClasses") {
    packagePrefix.set("some.cool.pkg")
  }
}

And this?
module some.coo.pkg.my.Module

@sgammon gotcha. I think the concern that you're calling out is that you cannot modify 3d party Pkl code (code that's hosted outside of a Kotlin codebase). That's indeed a valid concern, but I don't think this is the right solution to that problem.
I think what we should have instead is a way to map Pkl module names to Kotlin module names. The setting should look something like:
javaCodeGenerators {
  register("configClasses") {
    mappings.set(mapOf(
      "package://example.com/foo@1.0.0" to "com.example.foo"
    ))
  }
}

Generally, first-party Pkl code should have their module name match the expected package name.

authors: bioball sgammon


text:
[5.0] NSDecimalValue: Dont use .doubleValue for integer properties.

Decimal: Add an internal .int64Value and .uint64Value.

NSDecimalValue: For the .intValue and .uintValue properties
use a truncated value from .int64Value and .uint64Value.

Using .doubleValue lost information when returning large numbers

2e53 due to Double to Int conversion.

The returned values mostly match Darwin, moreso than just returning 0
when an exact conversion cannot be made.

(cherry picked from commit f369d31f91b267b2a57141eb610cf1cbda5bedde)

@swift-ci test

@swift-ci test and merge

repo: apple/swift-corelibs-foundation


text:
Fix TestNSData on Windows

In order to delete a directory on Windows, all handles to its children must be
closed.

@swift-ci please test


text:
add .spi.yml

Add .spi.yml for Crypto target so that DocC documentation is generated.

Could we incorporate 283 to add Crypto Extras as well?

repo: apple/swift-crypto


text:
Fix iOS navigator not calculating height properly

Bug/issue #, if applicable: 93506244
Summary
Exposes the viewport height as a Custom CSS Property, so it can be used for CSS calc expressions, instead of using 100vh units.
This is to fix a mis-calculation on iPads, where the height of the top and bottom safari navbars are not accounted for in the height of the navigator
Dependencies
NA
Testing
Please test this on Android, Safari and on Desktop on the popular browsers. This could not only add a performance degradation, but as you scroll, it may shift the height of the navigator, causing stuttering.
Steps:

Open a page with a navigator on iPad
Assert you can scroll all the way down and see the last navigator items, while still having the Safari nav items visible

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

@swift-ci test

repo: apple/swift-docc-render


text:
Add sidenav tag filtering

Bug/issue #, if applicable: 88001866
Summary
Allows filtering in the sidebar, using a pre-defined list of Tags
Dependencies
NA
Testing
Steps:

Serve a doccarchive that has articles or tutorials
Click on the filter and select Tutorial or Article tag
Assert only those are shown + their parents are open

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

@swift-ci test

@swift-ci test


text:
Remove special handling of the -experimental-lazy-typecheck flag

For now, the flag will simply be a frontend flag and the build system will pass it directly using -Xfrontend when appropriate.
Resolves rdar://117168788

@swift-ci please test

Related to https://github.com/apple/swift/pull/69259.

@swift-ci please test

@swift-ci please test Windows

@swift-ci please test

@swift-ci please test Windows

@swift-ci please test Windows

repo: apple/swift-driver


text:
[Incremental, Optimization] Defer use hashing when reading a ModuleDependencyGraph

When deserializing a ModuleDependencyGraph, don't record uses in a dictionary twice. Just save in an array while reading. On a project with 1,000 files, when rebuilding after just adding a newline to one file, swift-driver-new time in Instruments was reduced from 5.3 secs to 4.7.

@swift-ci please test

Thank you, @cltnschlosser !

@CodaFi , would you like to review before this lands?

I suspect the win you're seeing is because we're no longer hashing DependencyKeys. There's no double registration here AFAICT.
Not sure why you thought of double registration. But there was double dictionary access. Thanks for reviewing!

There is no double access. The mutation here goes through the modify coroutine. We don't lookup once to read and once to append https://github.com/apple/swift/blob/main/stdlib/public/core/Dictionary.swift#L893

There is no double access either. The mutation here goes through the modify coroutine. We don't lookup once to read and once to append https://github.com/apple/swift/blob/main/stdlib/public/core/Dictionary.swift#L893

I understand that. The other access is here: ```
let isNewUse = self.graph.nodeFinder
.record(def: dependencyKey, use: self.allNodes[useID])
inside `record`. There was one dictionary access while reading, and another while finalizing.

authors: CodaFi davidungar


text:
Create 0045-remove-falliable-initilizer.md

Here is my proposal for removing fallible initializers.
https://github.com/jcampbell05/swift-evolution/blob/master/proposals/0045-remove-falliable-initilizer.md

The discussion for this is still ongoing, almost everyone is against it.

@jcampbell05 @lattner . +1 one for removing. They really have no purpose with availability of throwing init.

Some people liked the alternative proposal, is it worth pursuing that?

authors: dileping jcampbell05 lattner

repo: apple/swift-evolution


text:
Leading whitespace on newlines

Xcode's default spacing for newlines ( "empty" lines) contains 4x leading spaces. I tried digging through all the options & wasn't able to find a way to use swift-format, & maintain that behavior.
Example Code (Take a look at lines 2 & 4.
    func XO() {}

    func xp() {}
    //What I Want

Tracked in Apple’s issue tracker as rdar://126948449

+1, I realize this isn't a priority for open source, but it would be great to be able to create a "doesn't conflict with control-I in Xcode" `.swift-format"

+1, Compatibility with Xcode formatting would be great! Other people on my team work in xcode, and I don't really want to force them to change formatting rules

authors: DenZhigalov GeorgeLyon SENTINELITE ahoppen

repo: apple/swift-format


text:
Remove calls to FileHandle.synchronizeFile().

Standard output does not support fsync, which is the system
call that this boils down to on Linux (and presumably on Darwin
as well). In both cases, fsync sets errno to EINVAL, but
unfortunately Foundation's behavior differs per platform: on
Darwin the error is silently swallowed, but on Linux it traps.

cc: @dylansturg


text:
Avoid querying funciton debug info while building the symbol table.

This fixes a testcase breakage introduced by 7eba1df14c04ce4ce812e5f0ab8f108899550001.
rdar://problem/48580111

@swift-ci test and merge

repo: apple/swift-lldb


text:
Cherry-pick the fixes for SR-7388 to swift-4.1-branch

This fixes two bugs that prevent LLDB and the REPL from working properly on Linux. For Darwin this is NFC.

Fix the installation path of the clang resource directory (Linux only!)
Derive LangOpts.EnableObjCInterop every time the triple is changed

https://bugs.swift.org/browse/SR-7388

@swift-ci test

The Linux test failed because there is also a build-script change necessary to pass the path to Foundation and libdispatch to lldb's dotest.py on Linux.

test with https://github.com/apple/swift/pull/16424
@swift-ci test

@swift-ci test

This time I forgot the "test with" command...

test with https://github.com/apple/swift/pull/16424
@swift-ci test


text:
Really turn off all the ObjC runtime ISA expression logs if not verbose.

We apparently only turned off one of them.

@swift-ci please test and merge

Should we have the same change upstream? If we do already, I apologize for the noise.

My bad.  I'll cancel this an do it through llvm.org.

IIRC Adrian did a similar change so you might end up just back porting it.
Regardless, thanks for doing this :) [as I wanted to do it for a while]

authors: dcci jimingham


text:
empty: For testing CI

Motivation
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications
[Describe the modifications you've made.]
Result
[After your change, what will change.]
Test Plan
[Describe the steps you took, or will take, to qualify the change - such as adjusting tests and manual testing.]

@swift-server-bot compat-test-full

❯ less ci.log | grep "Test Case"
Test Case 'CompatibilityTest.testAWSLambdaRuntime' started at 2023-09-18 13:34:06.567
Test Case 'CompatibilityTest.testAWSLambdaRuntime' passed (10.254 seconds)
Test Case 'CompatibilityTest.testAzureIOTIdentityService' started at 2023-09-18 13:34:16.821
Test Case 'CompatibilityTest.testAzureIOTIdentityService' passed (11.969 seconds)
Test Case 'CompatibilityTest.testBox' started at 2023-09-18 13:34:28.790
Test Case 'CompatibilityTest.testBox' failed (138.455 seconds)
Test Case 'CompatibilityTest.testCiscoMindmeld' started at 2023-09-18 13:36:47.245
Test Case 'CompatibilityTest.testCiscoMindmeld' passed (10.423 seconds)
Test Case 'CompatibilityTest.testCloudHypervisor' started at 2023-09-18 13:36:57.667
Test Case 'CompatibilityTest.testCloudHypervisor' passed (16.408 seconds)
Test Case 'CompatibilityTest.testDiscourse' started at 2023-09-18 13:37:14.076
Test Case 'CompatibilityTest.testDiscourse' passed (82.009 seconds)
Test Case 'CompatibilityTest.testGithub' started at 2023-09-18 13:38:36.085

Will put just Github in and see how it goes.

@swift-server-bot compat-test-full

@swift-server-bot compat-test-full

@swift-server-bot compat-test-full

OK, so compiling the generated code, but excluding Github, takes around 22min: https://ci.swiftserver.group/job/swift-openapi-generator-compatibility-test-prb/31/.
So this is probably what we want for the opt-in pipeline.

repo: apple/swift-openapi-generator


text:
[SR-13346] Swift Package Manager Security Risk

Previous ID
SR-13346

Radar
rdar://problem/66586184

Original Reporter
kanecheshire (JIRA User)

Type
Bug

Environment
Should note that while I do have Xcode 12 betas installed, Xcode on the machine where I've reproduced this is installed from the App Store and xcode-select is using that version, not Xcode 12.

Additional Detail from JIRA

Votes
2

Component/s
Package Manager

Labels
Bug

Assignee
None

Priority
Medium

md5: cea434d0fea89ad03bb6f84afcc6d96f

Issue Description:
Sorry if this isn't the correct place to report this but there's no issue tracker on the SPM repo and after writing a post about this issue someone pointed me to this Jira instance to report it here.
I've found an easy way for anyone to execute arbitrary code via Swift Package Manager in the Package.swift file:
let package: Package = {
  doSomething()
  return Package(...)
 }()
func doSomething() {
// Run any code in here as soon as the package is resolved, including anything from AppKit like NSWorkspace, or Foundation like FileManager, Process or URLSession.
}

The above code compiles and resolves correctly as a Package.swift file, and if hosted remotely somewhere like GitHub, the doSomething() function will be called as soon as it's resolved. Currently, that means in Xcode 11 it will call doSomething() silently before you've even finished adding the dependency since Xcode clones the repo and resolves the package before adding it to a target.
Example repo: https://github.com/KaneCheshire/maleficent To reproduce:
git clone https://github.com/KaneCheshire/maleficent
cd maleficent
swift build
# note that `say` will speak a greeting to you by name

I have no idea the best way to make this safer at a SPM level, but since it provides a way for people to silently execute code at the same privilege level that Xcode has (since it's running inside an Xcode process) I wanted to raise the issue asap.

Is this still a security issue / vulnerability and that is why it is open?
I found it here [ https://medium.com/@KaneCheshire/swift-package-manager-is-a-security-risk-4d13f3a7bc3b ] ... it makes me hesitant to use SwiftPM...

https://medium.com/@KaneCheshire/swift-package-manager-is-a-security-risk-4d13f3a7bc3b

I made a small "malicious" package to test this bug. It seems that it does not manage to start any processes when resolved, fetched or when the SwiftPM executable or Xcode macOS App is run. If I understand correctly, the Package.swift itself is executed in a sandbox?
Print statements are shown as warnings when the package is installed using SwiftPM, however I see no such warning in Xcode.

authors: benjiwolff swift-ci thereaderfortyfive

repo: apple/swift-package-manager


text:
[PackageGraph] Fix package name validation for product target dependencies

The package name validation was wrong for product target dependencies
(.product(name: "<product>", package: "<package>")) as we were looking
up the package using its identity instead of the name. This completely
breaks package loading in 5.2 if a package wants to use a product from
a dependency that doesn't match the package name of that dependency.
rdar://problem/59744465

@swift-ci smoke test


text:
PubgrubDependencyResolver: fix typo in child scope description

prefetcing -> prefetching

@swift-ci smoke test


text:
[perf_test] Update performance smoke test project configurations

All of the projects in smoke test compiler performance currently fail.

@swift-ci please test source compatibility

@swift-ci please smoke test compiler performance

@swift-ci please test source compatibility

@swift-ci please smoke test compiler performance

@swift-ci please test

@swift-ci please test

repo: apple/swift-source-compat-suite


text:
[NFC] Rename SyntaxData.replacingChild(_:at:arena:)

to SyntaxData.replacingChild(at:with:arena). This is consistent with RawSyntax.

@swift-ci Please test

repo: apple/swift-syntax


text:
[SE-0407] Add missingConformancesTo argument to MemberMacro expansion operation

Stage in an entrypoint for member macros that allows them to learn about which conformances that they've asked about are "missing", meaning that they are not present on the type (ignoring those that would be generated by an extension macro). This information is equivalent to the information provided to extension macros, although the member macro itself cannot create the conformance.

@swift-ci please test


text:
Typed throws

Parsing of typed throw specifiers. See https://github.com/apple/swift/pull/68629 for more information.

All of this should be behind an experimental language feature.

Great point, and it was super-easy to do. Thanks!

https://github.com/apple/swift/pull/68629
@swift-ci please test

https://github.com/apple/swift/pull/68629
@swift-ci please test

https://github.com/apple/swift/pull/68629
@swift-ci please test Windows

https://github.com/apple/swift/pull/68629
@swift-ci please test

https://github.com/apple/swift/pull/68629
@swift-ci please test

https://github.com/apple/swift/pull/68629
@swift-ci please test

https://github.com/apple/swift/pull/68629
@swift-ci please test Windows

https://github.com/apple/swift/pull/68629
@swift-ci please test

https://github.com/apple/swift/pull/68629
@swift-ci please test Windows

https://github.com/apple/swift/pull/68629
@swift-ci please test Linux

I’ve got a couple more comments in-line. On a higher level, the following still needs to be done.

The following should be SPI

The ThrownTypeClauseSyntax type
SyntaxAnyVisitor.visit(ThrownTypeClauseSyntax)
SyntaxAnyVisitor.visitPost(ThrownTypeClauseSyntax)
SyntaxEnum.thrownTypeClause
SyntaxKind.thrownTypeClause
SyntaxRewriter.visit(ThrownTypeClauseSyntax)
SyntaxVisitor.visit(ThrownTypeClauseSyntax)
SyntaxVisitor.visitPost(ThrownTypeClauseSyntax)
The initializers of AccessorEffectSpecifiersSyntax etc. that take ThrownTypeClauseSyntax
unexpectedBetweenThrowsSpecifierAndThrownError and unexpectedAfterThrownError

Provide unexpectedAfterThrowsSpecifier that concatenates unexpectedBetweenThrowsSpecifierAndThrownError, thrownError, and unexpectedAfterThrownError
Ensure that all the expression tests like [() try(MyError) async -> Void]() don’t parse once #2255 is fixed

We just discussed the design of the syntax tree and I think the tree should be structured as follows:

The effect specifier nodes contain a throwsClasue: ThrowsClauseSyntax child, instead of throwsSpecifier and thrownType
The ThrowsClauseSyntax has 4 children

throwsSpecifier: TokenSyntax (which can be either throws or rethrows)
leftParent: TokenSyntax?
thrownType: TypeSyntax?
rightParen: TokenSyntax?

authors: DougGregor ahoppen


text:
[NFC?] Clean up use-of-temporary in closure diagnostics

Rebases and updates #10196, because the fix is valuable to have.
The recursive lambda used to return StringRefs pointing inside out-streams with backing buffers that were local to itself.  Instead, take the stream as a parameter and write into that directly.
Should be NFC, but the base patch claims that test/Constraints/tuple_arguments.swift occasionally fails because of this.

@swift-ci please smoke test

To add more info to the original issue:
I had swift-4.0-branch checked out and for me the test was failing 100% on Release + Debug Info + Asserts, as well as on Debug + Asserts. I used clang 4.0 for the compilation.

authors: CodaFi acpopescu

repo: apple/swift


text:
[WIP][CSSolver] Attempt to remove type-checker hacks

Attempt to simplify the solver by removing (and replacing) most of the hacks.

@swift-ci please test source compatibility

@swift-ci please smoke test macOS platform

apple/swift#15311
@swift-ci please smoke test macOS platform

@swift-ci please smoke test macOS platform

@swift-ci please smoke test macOS platform


text:
improved String and StringUTF16, reapplied mistakenly stashed changes…

@gribozavr FYI

LGTM, merging!

authors: gribozavr shawnce


text:
Propagate tensorflow convention to the closure expressions as needed.

This fixes https://bugs.swift.org/browse/SR-8487.

@swift-ci please test tensorflow

@swift-ci please test tensorflow Linux
@swift-ci please test tensorflow macOS

@swift-ci please test tensorflow Linux CPU

@swift-ci please test tensorflow Linux CPU

The macOS build machine is offline. @shahmishal Any idea why? https://ci-external.swift.org/computer/macOS-tensorflow/

@rxwei It ran out of space, going to send you an email with more detail.

@bgogul You can only run one trigger per comment message.

Thanks Mishal!
We shouldn't let macOS CI block merges until the disk space issue is fixed.

authors: bgogul rxwei shahmishal


text:
[Mangling] Mangle types as generic only when they have generic arguments.

The isSpecialized() check didn’t account for the possibility that a
typealias in a parent of a nominal type could be non-generic when the
parent declaration was generic, leading to incorrect mangling that stated
that they were generic but had no generic arguments.
Fixes SR-8930 / rdar://problem/45216653 and rdar://problem/45813164.

@swift-ci please test and merge


text:
test: make make-unreadable Python 3 friendly

Make make-unreadable python 3 compatible.  POINTER is now in
ctypes rather than in ctypes.wintypes.  Use the new location which
is compatible across python 2 and python 3.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @tachoknight @drexin

@swift-ci please test


text:
[TypeChecker] Add error message when accessing a type's destructor

Trying to access a destructor using foo.deinit currently crashes the compiler. This pull requests makes the compiler create error messages instead.

Resolves SR-2652/SR-3043.

@swift-ci please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - b0e67da844d7b8b17b5149af28896d4a9eb8a246
Test requested by - @ahoppen

I can't really make sense of the build failure on Linux in llbuild. Could someone help me spot what the issue is? (I don't have a Linux installation locally and all my Mac tests pass). Will run the CI again in the meantime to check if it was just a temporary problem.
@swift-ci please test linux platform

OK, seems like the Linux test failures were nondeterministic or related to something else

This LGTM, thanks!

authors: DougGregor ahoppen swift-ci


text:
[build-windows] Minor cleanups

Cleans up the utils/build-windows.bat script for correctness, efficiency and robustness.

Don't download ICU since it’s not required for Swift standard library now and we don’t build Foundation in the script.
Replace hard-coded C: with %SystemDrive%.

Resolves SR-NNNN.

@swift-ci please smoke test

@swift-ci please smoke test

authors: compnerd stevapple


text:
Windows: add wincred.h to the modulemap

This adds the Windows credentials API to the modulemap for the Windows SDK module.

@swift-ci please test


text:
swift-plugin-server: remove standard headers

Redefine the types rather than use the standard headers due to the circular dependency between Darwin and Swift.

@swift-ci Please smoke test


text:
Forwarding force unwrap 6.0

(includes https://github.com/apple/swift/pull/73411 as a prerequisite)
Explanation: Changes the handling of the x! force unwrap operator on Optional to be any of borrowing, consuming, or mutating based on the use of the unwrapped value.
Scope: Noncopyable stdlib adoption work.
Issue: rdar://127459955
Original PR: https://github.com/apple/swift/pull/73531
Risk: Low. The change only affects the ! operator and should be a safe extension of its behavior for noncopyable Optionals.
Testing: Swift CI, test case from bug report
Reviewer: TBD

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test Windows


text:
[stdlib] Add documentation for ===

This behavior is documented in Equatable, but it would help to have it documented here as well.

@gribozavr I've amended this to add the backticks. Thanks for reviewing

Ah ok, first time I've seen this. Thanks!

authors: daniel-beard raylillywhite


text:
Fix generic super convenience init delegation

Fixes rdar://problem/31000248.

@swift-ci Please smoke test

@swift-ci Please smoke test Linux

@swift-ci Please smoke test Linux


text:
Trying to go to the last row doesn't directly take me there.

On macOS High Sierra in the Debug build.
In the cats and dogs dataset, there are 24769 rows (0-indexed). When I try to go to row number 24769 (which doesn't exist), it takes me to row number 24712. I feel like if the user enters a row number that's greater than the last row number (24768), there should be a friendly pop up message that tells them that it's an invalid row number and that they should enter a row number less than 24768. Repro video attached.
last-row.zip

I think we should go to the last row if a user enters a number over that (and probably go to the first row if a negative number is entered?).

authors: shantanuchhabra znation

repo: apple/turicreate


text:
[Feature]: data source for Directus / NocoDB / Grist

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
To build an integration of Directus / NocoDB / Grist with their OpenAPI Specification
https://docs.directus.io/reference/system/server/#get-openapi-specification
Why should this be worked on?
To make it smoother like on n8n when we need to bind the datasource with the rest api .
And also I was browsing in a subreddit "selfhosted" and I saw Budibase asking to build data source for Grist too.

I agree that these integrations would make a difference among these tools in terms of productivity and ux. See tooljet with appwrite for instance.

repo: appsmithorg/appsmith


text:
[Bug]: The home page icon is displayed for both old home page and new home page in roles page

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The home page icon is displayed for both old home page and new home page in roles page

Steps To Reproduce

Add a new app
Now the page one is a home page by default
Generate a new crud page(thats page 2)
Now set page2 as home page
Now go to a roles tab
Check the Pages the home icon is displayed for both the pages

Public Sample App
No response
Issue video log
No response
Version
localhost

This is expected. We show you a union of edit and view pages and hence you would see both the home pages aka home page in edit mode and home page in view mode

authors: RakshaKShetty ankitakinger


text:
[Bug]: MySQL: test datasource shows pass when port number field is left blank

Is there an existing issue for this?

[X] I have searched the existing issues

Description
https://theappsmith.slack.com/archives/C040LHZN03V/p1692927779184399
Steps To Reproduce
https://theappsmith.slack.com/archives/C040LHZN03V/p1692927779184399
Public Sample App
No response
Environment
Production
Issue video log
https://theappsmith.slack.com/archives/C040LHZN03V/p1692927779184399
Version
cloud, self hosted

not reproducible anymore

authors: rohan-arthur sumitsum


text:
[Bug]: 'Required' Field Validation in JSON Form Not Working Correctly If Fields Are Hidden

Is there an existing issue for this?

[X] I have searched the existing issues

Description
If we have 2 fields and want to alternatively make them required and visible, based on the selection from a dropdown, the Required validation does not work as expected and the Submit button stays disabled.
It seems that once a field was marked as Required and it gets programmatically hidden, the Required mark does not get removed programmatically as it should (it ignores the JS condition).
See the screen recording and sample app for more details.
Front thread
Steps To Reproduce

Use a JSON Form having one select "Type" (options Red and Blue) and two Input widgets labeled "Red" and "Blue".
If the value of "Type" is "Red", then the "Red" field should be visible and required; also the "Blue" field should be non-required and hidden.
If the value of "Type" is "Blue", then the "Blue" field should be visible and required; also the "Red" field should be non-required and hidden.
For the Red Field in the required js put {{formData.Type == "Red"}} and visibility js put {{formData.Type == "Red"}}
For the Blue Field in the required js put {{formData.Type == "Blue"}} and visibility js put {{formData.Type == "Blue"}}
Now if the Type is selected as "Red", only the Red field should be visible and required, but this is not happening if we switch between red and blue in the type, it also considers both fields required validation.

Public Sample App
https://app.appsmith.com/app/jsonform-validation-required/page1-65280f6880b32971dff37e3c
Environment
Production
Issue video log
https://www.loom.com/share/50c7b4b6562544dfa598216aa82094fc
Version
Cloud v1.9.40

It still does not work after resetting the form (see sample app and reproduction steps in the screen recording).


text:
chore: WDS elevation system fine-tuning

Follow-up to #29391
Consider:

lightness checks for seed properties (e.g. light seeds in light mode)
having a condition for the visibility of shadows
extending this to more semantics (accent, neutral etc.)

Hey team! Please add your planning poker estimate with Zenhub @jsartisan @KelvinOm


text:
[Feature]: Hosted API's

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Allow one to build actual API endpoints within Appsmith for other external applications or platforms to retrieve data from or "do" things. JS, Python, or my favorite, .NET/C#.
Why should this be worked on?
This is the next level of full application development!

Hey @tlawrence-TAG,
We can do most of this with Workflows, but it's my understanding that you'd like workflows that can return data. Is that correct?

@tlawrence-TAG  - We're working on a feature that can do this, would you be open to getting on a short call with me to better understand your use case?

Correct
Todd Lawrence
Chief Technology Officer
E: @.***
From: Laguna @.>
Sent: Monday, 30 September, 2024 1:43 PM
To: appsmithorg/appsmith @.>
Cc: Todd Lawrence @.>; Mention @.>
Subject: Re: [appsmithorg/appsmith] [Feature]: Hosted API's (Issue #36620)
Hey @tlawrence-TAGhttps://github.com/tlawrence-TAG,
We can do most of this with Workflows, but it's my understanding that you'd like workflows that can return data. Is that correct?
—
Reply to this email directly, view it on GitHubhttps://github.com/appsmithorg/appsmith/issues/36620#issuecomment-2383914513, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATUVW7NEIG3K74P3CFJUQJ3ZZGLU3AVCNFSM6AAAAABPD6RB6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBTHEYTINJRGM.
You are receiving this because you were mentioned.Message ID: @.@.>>
Please refer to our Company’s Email Disclaimerhttps://www.amherst.com/amherst-email-disclaimer for important disclosures regarding this electronic communication.

Absolutely
Todd Lawrence
Chief Technology Officer
E: @.***
From: Akhil Varma @.>
Sent: Monday, 30 September, 2024 1:46 PM
To: appsmithorg/appsmith @.>
Cc: Todd Lawrence @.>; Mention @.>
Subject: Re: [appsmithorg/appsmith] [Feature]: Hosted API's (Issue #36620)
@tlawrence-TAGhttps://github.com/tlawrence-TAG - We're working on a feature that can do this, would you be open to getting on a short call with me to better understand your use case?
—
Reply to this email directly, view it on GitHubhttps://github.com/appsmithorg/appsmith/issues/36620#issuecomment-2383918711, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATUVW7JJEGRHLXMZPGPVJULZZGL6LAVCNFSM6AAAAABPD6RB6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBTHEYTQNZRGE.
You are receiving this because you were mentioned.Message ID: @.@.>>
Please refer to our Company’s Email Disclaimerhttps://www.amherst.com/amherst-email-disclaimer for important disclosures regarding this electronic communication.

Great. Can you suggest a good time that works for you? Alternatively, here's my Calendly.

Customer Request Details - Notion